[mythtv-commits] Ticket #5066: myth_system broken by MYTH_SYSTEM_DONT_BLOCK_PARENT

MythTV mythtv at cvs.mythtv.org
Sat Mar 29 22:44:45 UTC 2008


#5066: myth_system broken by MYTH_SYSTEM_DONT_BLOCK_PARENT
---------------------------------+------------------------------------------
 Reporter:  pebender at san.rr.com  |       Owner:  ijr       
     Type:  defect               |      Status:  new       
 Priority:  minor                |   Milestone:  unknown   
Component:  mythtv               |     Version:  0.21-fixes
 Severity:  low                  |     Mlocked:  0         
---------------------------------+------------------------------------------
 With the addition of MYTH_SYSTEM_DONT_BLOCK_PARENT in changesets 16153 and
 16155, myth_system does not all input events when
 MYTH_SYSTEM_DONT_BLOCK_PARENT is used.

 You can see this by using LIRC to exit mythbrowser lauched by either
 MythBrowser or MythNews. Not only does mythbroswer process the Esc, but
 mythmainwindow does as well. This causes MythBrowser or MythNews to
 incorrectly exit when mythbroswer exits.

 I believe that the problem is caused by when the qApp->processEvents()
 call is made in myth_system. Since it is not called after the child
 process exists, I believe there is a chance of a race condition that
 results in myth_system exiting and re-enabling input processing before the
 LIRC Esc has been discarded.

 If qApp->processEvents() is called one last time after the child process
 exits, then the LIRC Esc event is flushed. This can be done by moving the
 location of the call.

 The attached patch makes the change and fixes the problem for me.

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


More information about the mythtv-commits mailing list