Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: MythTV: Commits

mythtv commit: r23051 by danielk

 

 

MythTV commits RSS feed   Index | Next | Previous | View Threaded


mythtv at cvs

Jan 1, 2010, 8:27 AM

Post #1 of 1 (205 views)
Permalink
mythtv commit: r23051 by danielk

Author: danielk
Date: 2010-01-01 16:27:46 +0000 (Fri, 01 Jan 2010)
New Revision: 23051
Changeset: http://svn.mythtv.org/trac/changeset/23051

Modified:

trunk/mythtv/libs/libmyth/mythcontext.cpp
trunk/mythtv/libs/libmyth/mythcontext.h

Log:

Refs #7603. Fixes segfault in mythcontext popup handling.

This drops the old UI popups and uses only MythUI pop-ups. The MythUI pop-ups do not try to launch from the calling thread anymore but instead an event is dispatched to an object whose event handler is in the MythUI thread. This simplifies the code and avoids all the problems, including segfaults, which occur when you try to use MythUI outside the MythUI thread.

Note: When MythProto is used within the MythUI thread, the popup will not be shown when it should be. This is pre-existing behaviour and is inherent in the design of MythUI. But a previous commit causes such usage of MythProto to print a warning message:
SendReceiveStringList(...) called from UI thread
These should be fixed, as the have potential to make the frontend appear to be hung for minutes at a time.


_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits

MythTV commits RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.