Opened 11 years ago

Closed 11 years ago

#5602 closed defect (fixed)

Re-enable MediaMonitor events after library re-org

Reported by: Nigel Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: mythtv Version: unknown
Severity: medium Keywords:
Cc: Ticket locked: no

Description

First hacky attempt at re-enabling. Moves handler registration into the monitor, and attempts to call Plugins' handlers directly from there, but I think this has problems, because the monitor isn't the main thread, and invoking Plugin menus/dialogs outside of that may cause problems.
Only other way I can think of doing this would be via events. Sort of like before, but with all the media dependencies at sender and receiver of the event, and not in the middle (libmythui/MythMainWindow)

Attachments (2)

mediamon.patch (13.8 KB) - added by Nigel 11 years ago.
mediamon.2.patch (14.9 KB) - added by Nigel 11 years ago.
Uses events, correctly jumps around in menus

Download all attachments as: .zip

Change History (3)

Changed 11 years ago by Nigel

Attachment: mediamon.patch added

Changed 11 years ago by Nigel

Attachment: mediamon.2.patch added

Uses events, correctly jumps around in menus

comment:1 Changed 11 years ago by Nigel

Resolution: fixed
Status: newclosed

(In [18122]) Re-enable media monitor after library circ.-deps fix. Moves media event processing from libmythui, to an extra event filter. Closes #5602.

Note: See TracTickets for help on using tickets.