Sorry, I seem to have lost the original mail, and as Gmane is dead I had to review using Nicolas's mail.
On Tue, Aug 2, 2016 at 8:14 AM Nicolas George <geo...@nsup.org> wrote: > Le primidi 11 thermidor, an CCXXIV, sebechlebsky...@gmail.com a écrit : > > diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h > > index 36f7935..39106ee 100644 > > --- a/libavcodec/avcodec.h > > +++ b/libavcodec/avcodec.h > > @@ -5949,6 +5949,80 @@ void av_bsf_free(AVBSFContext **ctx); > > */ > > const AVClass *av_bsf_get_class(void); > > > > +/** > > + * Structure for chain/list of bitstream filters. > > + * Empty list can be allocated by av_bsf_list_alloc(). > > + */ > > +typedef struct AVBSFList AVBSFList; > > + > > +/** > > + * Allocate empty list of bitstream filters. > > + * The list must be later freed by av_bsf_list_free() > > + * or finalized by av_bsf_list_finalize(). > > + * > > + * @return pointer to AVBSFList on success, NULL in case of failure > For Doxygen documentation, capitalize the first letter of the descriptions in @return's and @param's, and @ref-erence types, structs, and enums. (For functions, an empty set of parentheses is enough.) The same rules apply to all the new functions you've added. @return Pointer to @ref AVBSFList on success, NULL in case of failure. > > + */ > > +AVBSFList *av_bsf_list_alloc(void); > > + > > +/** > > + * Free list of bitstream filters. > > + * > > + * @param lst pointer to pointer returned by av_bsf_list_alloc() > @param lst Pointer to pointer returned by av_bsf_list_alloc() > + */ > > +void av_bsf_list_free(AVBSFList **lst); > > + > > +/** > > + * Append bitstream filter to the list of bitstream filters. > > + * > > > + * @param lst list to append to > List > > + * @param bsf AVBSFContext to be appended > @ref AVBSFContext > > + * > > + * @return >=0 on success, negative AVERROR in case of failure > > + */ > > +int av_bsf_list_append(AVBSFList *lst, AVBSFContext *bsf); > > + > > +/** > > + * Finalize list of bitstream filters. > > + * > > + * This function will transform AVBSFList to single AVBSFContext, > @ref's > > + * so the whole chain of bitstream filters can be treated as single > filter > > + * freshly allocated by av_bsf_alloc(). > > > + * The AVBSFList structure is destroyed after this call and must not > Ditto > > + * be accessed. > > + * > > + * @param lst AVBSFList structure to be transformed > > > + * @param bsf[out] pointer to be set to newly created AVBSFContext > structure > @param[out] bsf Same applies to rest of the file. > > + * representing the chain of bitstream filters > > + * > > + * @return >=0 on success, negative AVERROR in case of failure > > + */ > > +int av_bsf_list_finalize(AVBSFList **lst, AVBSFContext **bsf); > > + > > +/** > > + * Parse string describing list of bitstream filters and create single > > + * AVBSFContext describing the whole chain of bitstream filters. > > + * Resulting AVBSFContext can be treated as any other AVBSFContext > freshly > > + * allocated by av_bsf_alloc(). > > + * > > + * @param str string describing chain of bitstream filters in > format > > > + * bsf1[=opt1=val1:opt2=val2][,bsf2] > Use Markdown backtick syntax to make it display in monospace font `bsf[=opt1=val1:opt2=val2][,bsf2]` > > + * @param bsf[out] pointer to be set to newly created AVBSFContext > structure > > + * representing the chain of bitstream filters > > + * > > + * @return >=0 on success, negative AVERROR in case of failure > > + */ > > +int av_bsf_list_parse_str(const char *str, AVBSFContext **bsf); > > + > > +/** > > + * Get null/pass-through bitstream filter. > > + * > + * @param bsf[out] pointer to be set to new instance of pass-through > > + * bitstream filter > + * > > + * @return > > + */ > > +int av_bsf_get_null_filter(AVBSFContext **bsf); > > + > > /* memory */ > > > > /** > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel