[FFmpeg-devel] [PATCH v5 1/7] libavformat/oggdec.h: Document packet function return value.

[Romain Beauxis]

Le mer. 14 mai 2025 à 17:32, Michael Niedermayer <michael at niedermayer.cc> a
écrit :
>
> On Fri, May 09, 2025 at 06:43:21PM -0500, Romain Beauxis wrote:
> > ---
> >  libavformat/oggdec.h | 6 ++++++
> >  1 file changed, 6 insertions(+)
> >
> > diff --git a/libavformat/oggdec.h b/libavformat/oggdec.h
> > index 43df23f4cb..5225b77a07 100644
> > --- a/libavformat/oggdec.h
> > +++ b/libavformat/oggdec.h
> > @@ -38,6 +38,12 @@ struct ogg_codec {
> >       *         -1 if an error occurred or for unsupported stream
> >       */
> >      int (*header)(AVFormatContext *, int);
> > +    /**
> > +     * Attempt to process a packet as a data packet
> > +     * @return < 0 (AVERROR) code or -1 on error
> > +     *         == 0 if the packet was a regular data packet.
> > +     *         == 0 or 1 if the packet was a header from a chained
bitstream.
> > +     */
> >      int (*packet)(AVFormatContext *, int);
> >      /**
> >       * Translate a granule into a timestamp.
> > --
>
> will apply, so the patchset becomes smaller

Thanks!

I'll address the comments on the vorbis diff.

Opus and flac could go in too if you want.

-- Romain