[Ffmpeg-devel] [PATCH] fix usage of various bitstream readers

[Michael Niedermayer]

Hi

On Mon, Aug 28, 2006 at 07:16:43PM +0200, Aurelien Jacobs wrote:
> On Mon, 28 Aug 2006 19:13:29 +0200
> Aurelien Jacobs <aurel at gnuage.org> wrote:
> 
> > On Mon, 28 Aug 2006 17:57:54 +0200
> > Michael Niedermayer <michaelni at gmx.at> wrote:
> > 
> > > Hi
> > > 
> > > On Mon, Aug 28, 2006 at 05:13:02PM +0200, Aurelien Jacobs wrote:
> > > > Hi,
> > > > 
> > > > This set of patch allows to use the A32_BITSTREAM_READER without any
> > > > regression (as well on x86 as on arm).
> > > > The first one moves get_bits_long(), show_bits_long() and check_marker()
> > > > to bitstream.h to avoid conflicts between different bitstream readers in
> > > > different codecs.
> > > > The second one forces the use of ALT_BITSTREAM_READER for the dv and flac
> > > > codec. dv.c don't compile with any other bitstream reader, and flac.c
> > > > crashes with other bitstream reader. It also forces the usage of
> > > > ALT_BITSTREAM_READER for codecs who needs ALT_BITSTREAM_READER_LE.
> > > > 
> > > > Ok to apply ?
> > > [...]
> > > > Index: libavcodec/bitstream.h
> > > > ===================================================================
> > > > --- libavcodec/bitstream.h	(revision 6102)
> > > > +++ libavcodec/bitstream.h	(working copy)
> > > > @@ -6,6 +6,8 @@
> > > >  #ifndef BITSTREAM_H
> > > >  #define BITSTREAM_H
> > > >  
> > > > +#include "avcodec.h"
> > > 
> > > why?
> > 
> > Because a call to av_log (using AV_LOG_INFO) is added to this file.
> > I've just found out that the same include was already present in this
> > file (but only conditionnaly), so I removed the conditionnal one.
> > Updated patch attached.
> 
> hum.... and here is the patch ;-)

isnt log.h enough?

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

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is