[mythtv-commits] Ticket #2909: Channel Scan pop-up deleted outside of Qt event thread

[MythTV]

#2909: Channel Scan pop-up deleted outside of Qt event thread
-------------------------+--------------------------------------------------
 Reporter:  sipsniffa    |        Owner:  danielk
     Type:  defect       |       Status:  new    
 Priority:  minor        |    Milestone:  0.21   
Component:  channelscan  |      Version:  head   
 Severity:  medium       |   Resolution:         
  Mlocked:  0            |  
-------------------------+--------------------------------------------------

Comment(by danielk):

 (In [14362]) Refs #2909. Refs #3326. Fixes Channel Scan popup segfault in
 mythtv-multirec.

 The problem when we applied this to trunk in [12624] & [12629] which
 caused the reversion in [12635] appears to have been the addition of some
 popupDone() signals when we were done with a MythPopupBox. This signal is
 interpreted in several places in playbackbox,cpp and once in
 welcomedialog.cpp to mean the popup was rejected not just done. I've added
 the return value to the popupDone signal so that these uses can be
 supported by simply checking that the return value is Rejected before
 interpreting the signal as a cancel.

 I'm fixing this in multirec is because the changes in multirec make the
 lack of thread safety in the popup handling cause a reproducable segfault
 every time you run the same channel scan twice. In the trunk this segfault
 is relatively rare.

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/2909#comment:13>
MythTV <http://svn.mythtv.org/trac>
MythTV