[mythtv-commits] Ticket #2738: Crash in MythThemedMenuPrivate::ReloadTheme()

[MythTV]

#2738: Crash in MythThemedMenuPrivate::ReloadTheme()
---------------------------------------------------------+------------------
 Reporter:  bolek-mythtv at curl.com (Boleslaw Ciesielski)  |        Owner:  ijr    
     Type:  defect                                       |       Status:  closed 
 Priority:  minor                                        |    Milestone:  unknown
Component:  mythtv                                       |      Version:  head   
 Severity:  medium                                       |   Resolution:  fixed  
---------------------------------------------------------+------------------
Changes (by paulh):

  * status:  new => closed
  * resolution:  => fixed

Comment:

 (In [12373]) This fixes a problem where exiting the Appearance settings
 sometimes causes a
 segfault. It actually only affects the classic menu layout but it was only
 by
 luck the DVR and default layouts worked.

 Basically this commit makes sure the AppearanceSettings dialog is deleted
 and
 cleaned up before doing the JumpTo("Reload Theme") so that when its
 detached
 from the MythMainWindow it doesn't send an ExitToMainMenuEvent which was
 causing all the problems.

 It also only reloads the theme if the user completed the settings dialog
 by
 pressing "Finish".

 Fixes #2860, #2580, #2738.

-- 
Ticket URL: <http://cvs.mythtv.org/trac/ticket/2738#comment:2>
MythTV <http://www.mythtv.org/>
MythTV