[mythtv] [mythtv-commits] Ticket #3666: DVD speed setting only works when using media monitoring

[Stuart Morgan]

On Tuesday 26 June 2007 00:28:07 Nigel Pearson wrote:
> Hell yes.
>
> Eskil's patches in '#2598 Use MythMediaMonitor in plugins'
> have most of the required changes
> (e.g. post _all_ media event types).

Great. I'd missed that.

> I also noted the other day that, if AutoLookupCD is set,
> databasebox.cpp does new and delete of a CdDecoder
> (with its CDDB lookup and MetaData creation), every 1000ms.
> Luckily, libcdaudio's CDDB lookup has caching.
> The problema is that my Qt-based Darwin lookup doesn't.

I've already re-written this into a single background thread with a single 
instance of cddecoder, using signals to inform playbackbox/databasebox of 
changes. It works well but I stopped work on it when I decided that the media 
monitor should probably be involved.

>
> But then I thought, we could even have the
> CDDB lookup in the MediaMonitor itself.
> And, then I decided my plans were bigger
> than my available free time, and stopped.

I can't see the purpose in having CDDB look up in the mediamonitor myself, but 
it can always been done at some point down the road if there is good reason 
for it.

> Stuart, do you have some time to work
> with me on these ideas/patches?

I don't have a great amount of free time because I've already dedicated most 
of it to re-writing the mythmusic playlist editor, but I can probably spare a 
little to see this idea through to completion.

-- 
Stuart Morgan