[mythtv-commits] mythtv commit: r23051 by danielk

[mythtv at cvs.mythtv.org]

      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.