[FFmpeg-devel] av_first_bitstream_filter in avcodec.h

Aurelien Jacobs aurel
Mon Sep 17 00:50:51 CEST 2007


On Mon, 17 Sep 2007 00:35:18 +0200
Michael Niedermayer <michaelni at gmx.at> wrote:

> 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

Applied.

Aurel




More information about the ffmpeg-devel mailing list