[mythtv-commits] Re: Ticket #341: Music database update does not reread metadata

MythTV mythtv at cvs.mythtv.org
Wed Oct 26 20:59:32 EDT 2005


#341: Music database update does not reread metadata
------------------------------------+---------------------------------------
 Reporter:  eskil <myth at eskil.org>  |        Owner:  ijr 
     Type:  patch                   |       Status:  new 
 Priority:  minor                   |    Milestone:  0.19
Component:  mythmusic               |      Version:  head
 Severity:  low                     |   Resolution:      
------------------------------------+---------------------------------------
Comment (by danielk):

 Oh, also you can't just delete QObject derived objects, you need to
 disconnect() any signals you have going to them, and then call their
 deleteLater() method. The disconnect() ensures that another MythTV
 thread doesn't call a method in the class after its memory is freed,
 and the deleteLater() ensures that that the Qt event thread doesn't
 send them any events to it after its memory is freed.

 Also, if you create any functions not taking a parameter please set
 declare their parameter list as (void), unless it is the destructor.
 This just makes it easier to grep for the deceleration and
 implementation of such a function.

 PS I'm already working on merging the busy.patch, so don't bother with
 that one.

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


More information about the mythtv-commits mailing list