[mythtv-commits] Ticket #4666: Leaking timers and deleteLater() for tv_play.cpp

MythTV mythtv at cvs.mythtv.org
Thu Feb 14 03:54:14 UTC 2008


#4666: Leaking timers and deleteLater() for tv_play.cpp
------------------------------------------------+---------------------------
 Reporter:  Shane Shrybman <gnome42 at gmail.com>  |       Owner:  ijr    
     Type:  patch                               |      Status:  new    
 Priority:  minor                               |   Milestone:  unknown
Component:  mythtv                              |     Version:  unknown
 Severity:  low                                 |     Mlocked:  0      
------------------------------------------------+---------------------------
 While tracking down a QTimer related segfault in the frontend that occurs
 when exiting the player a I noticed the timers aren't being disconnected
 or deleted. (patch attached)

 That lead to looking into the QT docs and the issues with signals and
 deleteLater(). The second patch implements deleteLater() for the TV class
 and changes all the users to use it instead of delete.

 The deleteLater() patch is partly for review/feedback. Since I'm not sure
 it's absolutely required but it seems like the safer thing to do.

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/4666>
MythTV <http://svn.mythtv.org/trac>
MythTV


More information about the mythtv-commits mailing list