[FFmpeg-devel] [PATCH v2] ogg/vorbis: implement header packet skip in chained ogg bitstreams.

[Michael Niedermayer]

On Sat, Jun 21, 2025 at 10:45:32AM +0200, Romain Beauxis wrote:
> Le dim. 15 juin 2025 à 00:57, Michael Niedermayer
> <michael at niedermayer.cc> a écrit :
> >
> > On Wed, Jun 04, 2025 at 11:58:52AM -0500, Romain Beauxis wrote:
> > > This is a redo of 574f634e49847e2225ee50013afebf0de03ef013 using a flat
> > > memory storage for the extradata.
> > >
> > > PR review comments addressed:
> > > * Use flat memory bytestream
> > > * Re-use existing xiph extradata layout
> > >
> > > ---
> >
> > >  libavcodec/vorbisdec.c                     | 42 ++++++++---
> > >  libavformat/oggparsevorbis.c               | 83 +++++++++++++++++++++-
> >
> > patches that change both libraries at the same time are suspect
> >
> > if one depends on changes in the other it needs
> > minor API version bump and seperate patches so extension of
> > API and use of it are properly tracked and testable
> 
> If I remember well, according to Andreas Rheinhardt there's no need
> for an API bump here since the patch is re-using existing extradata
> bitstream structures.

If there is really no API extension then the micro versions should be bumped
so a user knows if the specific version he uses has teh fix.
Also it may be usefull in bugreports about ogg to know if its prior or
after this change

thx

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

Awnsering whenever a program halts or runs forever is
On a turing machine, in general impossible (turings halting problem).
On any real computer, always possible as a real computer has a finite number
of states N, and will either halt in less than N cycles or never halt.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20250621/d18f9a01/attachment.sig>