[mythtv] [mythtv-commits] Ticket #7419: mythcommflag broken on HD-PVR x264 files

[Harley Peters]

On Sun, 25 Oct 2009 13:17:54 -0400
"Michael T. Dean" <mtdean at thirdcontact.com> wrote:

> On 10/25/2009 12:15 PM, Harley Peters wrote:
> > On Sat, 24 Oct 2009 22:40:49 -0600 John P Poet wrote:
> >   
> >> On Sat, Oct 24, 2009 at 9:11 PM, MythTV wrote:
> >>     
> >>> #7419: mythcommflag broken on HD-PVR x264 files
> >>>
> >>>  HD-PVR as my Tuner 1
> >>>
> >>>  Commercial flagging takes 100% CPU and prints
> >>>
> >>>  2009-10-23 17:03:05.969 AFD Error: Unknown decoding error
> >>>  2009-10-23 17:03:08.992 [h264 @ 0x7f481abbcd80]B picture before
> >>> any references, skipping
> >>>  2009-10-23 17:03:08.992 [h264 @
> >>> 0x7f481abbcd80]decode_slice_header error 2009-10-23 17:03:08.992
> >>> [h264 @ 0x7f481abbcd80]no frame! 2009-10-23 17:03:08.992 AFD
> >>> Error: Unknown decoding error
> >>>
> >>>  over and over.
> >>>       
> >> Those messages are normal.  The avformat library handles HD-PVR
> >> material well enough, but not perfectly.  You can basically treat
> >> those messages as warnings.
> > It would be nice if those "error" messages could be moved to a
> > different error level.
> > I have had to set my master backend's logging to none,general to
> > keep the log file from becoming huge over time.
> > This of course turns off most logging making seeing any real errors
> > (errors I can do something about) unlikely.
> >   
> 
> http://svn.mythtv.org/trac/ticket/6729
> 
> Mike
> _______________________________________________
> mythtv-dev mailing list
> mythtv-dev at mythtv.org
> http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev

Ok using nogeneral still leaves a lot of garbage in the logs. And also
turns off some logging I want.
The attached patch works well enough for me.
I don't expect it's good enough for general use but it's a better fit
for me.


--- libs/libmythtv/avformatdecoder.cpp.orig	2009-10-25
19:15:23.271761195 -0500 +++ libs/libmythtv/avformatdecoder.cpp
2009-10-25 22:23:34.900631324 -0500 @@ -120,7 +120,7 @@
             verbose_level = VB_IMPORTANT;
             break;
         case AV_LOG_ERROR:
-            verbose_level = VB_GENERAL;
+            verbose_level = VB_LIBAV;
             break;
         case AV_LOG_VERBOSE:
         case AV_LOG_INFO:
@@ -3984,7 +3984,7 @@
 
                     if (ret < 0)
                     {
-                        VERBOSE(VB_IMPORTANT, LOC_ERR +
+                        VERBOSE(VB_LIBAV, LOC_ERR +
                                 "Unknown decoding error");
                         have_err = true;
                         continue;

-------------- next part --------------
A non-text attachment was scrubbed...
Name: reduce_avformat_verbose_logging.patch
Type: text/x-patch
Size: 751 bytes
Desc: not available
URL: <http://mythtv.org/pipermail/mythtv-dev/attachments/20091025/19949c07/attachment.bin>