[mythtv] [mythtv-commits] Ticket #3666: DVD speed setting only works when using media monitoring
stuart at tase.co.uk
Tue Jun 26 06:31:50 UTC 2007
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
> 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.
More information about the mythtv-dev