[FFmpeg-devel] [PATCH] mpeg4videodec: silence "Invalid andinefficient vfw-avi packed B frames detected" warning

[Michael Niedermayer]

On Mon, Sep 09, 2013 at 09:22:07PM +0200, Reimar Döffinger wrote:
> On Sun, Sep 08, 2013 at 08:31:52PM -0400, Don Moir wrote:
> > >On Tue, Sep 03, 2013 at 04:35:35PM -0400, Don Moir wrote:
> > >>>Though for some reason this yet again seems to have degenerated into
> > >>>an IMO fairly pointless back and forth, with no attention
> > >>>or comments that would allow me to understand the issue and where I
> > >>>might have misunderstood the problem on the two suggestions I had
> > >>
> > >>Yes exactly. All I said was I would compile out the warnings if
> > >>good... just a switch mind you.. just a comment and not to generate
> > >>endless BS
> > >>
> > >>I expect the name calling to happen next :)
> > 
> > >>But since it's not something that could be done trivially with a
> > >>macro or so I'm not sure it's worth it, but it's interesting to
> > >>consider.
> > 
> > >Possibly not to trivial not sure. One thing is the fatals would have to be separated.
> > 
> > This seems to work for me but have not tried with GCC.
> > 
> > in log.h
> > 
> > #ifdef _NOWARNINGS
> >    inline void av_log(void *avc1, int level, const char *fmt,...){}
> > #else
> >    void av_log(void *avc1, int level, const char *fmt,...);
> > #endif
> > 
> > in log.c av_log (...) would need #ifdef
> > 
> > fatals still need to be separate
> 
> Yes, that's exactly where the problem lies.
> So you have to filter out those you do not want.
> And then it still needs to be more efficient than the function call.
> Then you have to be careful not to mess too much with the public
> headers.

> However it seems like the below would work, it would just needs
> someone to add properly to configure.

use below as you see fit

diff --git a/configure b/configure
index 4ff97b4..5d18f95 100755
--- a/configure
+++ b/configure
@@ -2482,6 +2482,9 @@ for opt do
         --enable-debug=*)
             debuglevel="$optval"
         ;;
+        --avlog-limit=*)
+            avloglimit="$optval"
+        ;;
         --disable-programs)
             disable $PROGRAM_LIST
         ;;
@@ -4819,6 +4822,7 @@ cat > $TMPH <<EOF
 #define EXTERN_ASM ${extern_prefix}
 #define SLIBSUF "$SLIBSUF"
 #define HAVE_MMX2 HAVE_MMXEXT
+#define AV_LOG_MIN_LEVEL $avloglimit
 EOF

 test -n "$assert_level" &&

 [...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Breaking DRM is a little like attempting to break through a door even
though the window is wide open and the only thing in the house is a bunch
of things you dont want and which you would get tomorrow for free anyway
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130909/e0e0a333/attachment.asc>