[FFmpeg-devel] av_first_bitstream_filter in avcodec.h

[Michael Niedermayer]

Hi

On Sun, Sep 16, 2007 at 11:57:28PM +0200, Aurelien Jacobs wrote:
> On Fri, 14 Sep 2007 10:00:37 +0200
> "Dean Pavlekovic" <dpavlekovic at gmail.com> wrote:
> 
> > Hi all,
> > 
> > Just one small thing, I've noticed that avcodec.h declares extern
> > 'av_first_bitstream_filter' variable which isn't defined nor
> > referenced anywhere else. However, there's 'first_bitstream_filter'
> > defined in bitstream_filter.c.
> > 
> > $ grep -r first_bitstream_filter *
> > libavcodec/avcodec.h:extern AVBitStreamFilter *av_first_bitstream_filter;
> > libavcodec/bitstream_filter.c:AVBitStreamFilter *first_bitstream_filter= NULL;
> > libavcodec/bitstream_filter.c:    bsf->next = first_bitstream_filter;
> > libavcodec/bitstream_filter.c:    first_bitstream_filter= bsf;
> > libavcodec/bitstream_filter.c:    AVBitStreamFilter *bsf=
> > first_bitstream_filter;
> > 
> > So is this 'av_' difference a typo or the first_bitstream_filter isn't
> > supposed to be exported, in which case the av_bitstream_filter
> > declaration in avcodec.h is superfluous? Or something else
> 
> Indeed, I think it's not supposed to be exported.
> Attached patch removes it.

ok

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

If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070917/986c8db3/attachment.pgp>