Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: MythTV: Dev

Re: [mythtv-commits] mythtv commit: r13670 by stuartm

 

 

MythTV dev RSS feed   Index | Next | Previous | View Threaded


mythtv at dsl

Jun 14, 2007, 10:00 AM

Post #1 of 2 (357 views)
Permalink
Re: [mythtv-commits] mythtv commit: r13670 by stuartm

mythtv [at] cvs wrote:
> Author: stuartm
> Date: 2007-06-14 14:17:05 +0000 (Thu, 14 Jun 2007)
> New Revision: 13670
> Changeset: http://cvs.mythtv.org/trac/changeset/13670
>
> Modified:
>
> trunk/mythplugins/mythmusic/mythmusic/decoder.cpp
>
> Log:
>
> Refs #3607
>
> Make the read metadata checks more robust. If reading from tags fails completely, try from the filename and if that fails log the error.
>
>

Should that not be 'if (!decoder)' in the last part of the patch?

Paul H.

_______________________________________________
mythtv-dev mailing list
mythtv-dev [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev


stuart at tase

Jun 14, 2007, 10:27 AM

Post #2 of 2 (335 views)
Permalink
Re: [mythtv-commits] mythtv commit: r13670 by stuartm [In reply to]

On Thursday 14 June 2007 18:00:13 Paul Harrison wrote:
> mythtv [at] cvs wrote:
> > Author: stuartm
> > Date: 2007-06-14 14:17:05 +0000 (Thu, 14 Jun 2007)
> > New Revision: 13670
> > Changeset: http://cvs.mythtv.org/trac/changeset/13670
> >
> > Modified:
> >
> > trunk/mythplugins/mythmusic/mythmusic/decoder.cpp
> >
> > Log:
> >
> > Refs #3607
> >
> > Make the read metadata checks more robust. If reading from tags fails
> > completely, try from the filename and if that fails log the error.
>
> Should that not be 'if (!decoder)' in the last part of the patch?

Heh, last part wasn't meant to be checked in at all. Thanks for noticing,
fixing that now.

--
Stuart Morgan
_______________________________________________
mythtv-dev mailing list
mythtv-dev [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev

MythTV dev RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.