[FFmpeg-devel] [PATCH] lavc/avcodec.h: document bitstream filter API

Stefano Sabatini stefasab at gmail.com
Fri Jul 5 11:06:01 CEST 2013


On date Friday 2013-07-05 01:52:28 +0200, Michael Niedermayer encoded:
> On Fri, Jul 05, 2013 at 01:19:44AM +0200, Stefano Sabatini wrote:
> > On date Thursday 2013-07-04 21:03:21 +0200, Michael Niedermayer encoded:
> > > On Thu, Jul 04, 2013 at 07:35:31PM +0200, Stefano Sabatini wrote:
> > > > On date Friday 2013-06-28 17:06:42 -0800, Lou Logan encoded:
> > > > > This is only a review of the language.
> [...]
> > > > + *
> > > > + * The filter will be accessible to the application code through
> > > > + * av_bitstream_filter_next() or can be directly initialized with
> > > > + * av_bitstream_filter_init().
> > > > + *
> > > > + * @see avcodec_register_all()
> > > > + */
> > > >  void av_register_bitstream_filter(AVBitStreamFilter *bsf);
> > > > +
> > > 
> > 
> > > > +/**
> > > > + * Create and initialize a bitstream filter context given a bitstream
> > > > + * filter name.
> > > > + *
> > > > + * The returned context must be freed with av_bitstream_filter_close().
> > > > + *
> > > > + * @param name    the name of the bitstream filter
> > > > + * @return a bitstream context if a corresponding filter was found and
> > > > + * successfully initialized, NULL otherwise.
> > > 
> > > "a bitstream context" ?
> > 
> > What's the question?
> 
> whats a bitstream context ?
> did you mean bitstream filter context >?

Fixed & pushed.

Thanks Michael and Lou for the review.
-- 
FFmpeg = Frenzy Fanciful Muttering Philosofic Emblematic Geisha


More information about the ffmpeg-devel mailing list