[mythtv-commits] Ticket #9537: MediaMonitor JumpTo plugin broken in commit ed4d961aed836fade236d006b2797f0c044970de

[MythTV]

#9537: MediaMonitor JumpTo plugin broken in commit
ed4d961aed836fade236d006b2797f0c044970de
-------------------------------------------------+-------------------------
 Reporter:  Lawrence Rust <lvr@…>                |          Owner:  stuartm
     Type:  Bug Report                           |         Status:
 Priority:  minor                                |  accepted
Component:  MythTV - General                     |      Milestone:  0.25
 Severity:  medium                               |        Version:  Trunk
 Keywords:                                       |  Head
                                                 |     Resolution:
                                                 |  Ticket locked:  0
-------------------------------------------------+-------------------------

Comment (by stuartm):

 ok, the reason why JumpTo is asynchronous is that it's frequently called
 from outside the UI Thread as is the case here. The callbacks are called
 outside the UI thread because the mediamonitor (correctly) sits in it's
 own thread.

 I think the best fix is what I proposed the other day on the mailing list,
 we get rid of the callbacks for plugins and instead use events, if the
 plugins receive MediaEvents then they can call JumpTo() only if required
 and JumpTo() can operate synchronously. A gCoreContext->IsUIThread() check
 in JumpTo() will handle that neatly.

-- 
Ticket URL: <http://code.mythtv.org/trac/ticket/9537#comment:2>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center