From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id 6AFDE4BC76 for ; Wed, 30 Apr 2025 00:17:50 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 871B668B118; Wed, 30 Apr 2025 03:17:47 +0300 (EEST) Received: from relay3-d.mail.gandi.net (relay3-d.mail.gandi.net [217.70.183.195]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 0DE04687B70 for ; Wed, 30 Apr 2025 03:17:41 +0300 (EEST) Received: by mail.gandi.net (Postfix) with ESMTPSA id 71CBF1FCE8 for ; Wed, 30 Apr 2025 00:17:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=niedermayer.cc; s=gm1; t=1745972260; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=OF9BUKe/eldr7r6+9fsdYoyvtvNjbtjaZWQXiJ9pJrY=; b=CC2ZiidOWqv1Z4KNogWa2bQE84A11YILEp4xVLRTDakKmAlMZQnwalcgjvr48kidqIMZEH W1+yoZb9zK+7PmrT5rSQaHcA1/Zcj0n+SmuqVNmt3UwkYzs4lUFgTA0XL6EgPLQTpDrDjZ 3mzUSn8V44ly8QOjBU5ysRjVCGrLKntGxeS0iaTNsyoqz6hawvywGSfWnDYvnOcpa4j1Dm 5oBa26pnXqkrGsYvfRph2a+hnXODCoXjYSl9tEWAy9oES2utMK+QddlK4flJSx+ihLPGNT IvoDDqDBUMYRbOH5hyJSGLnYHgsd3/Qz775Gka68A9PmNh7JhhDwAqKV+ttXSA== Date: Wed, 30 Apr 2025 02:17:39 +0200 From: Michael Niedermayer To: FFmpeg development discussions and patches Message-ID: <20250430001739.GV4991@pb2> References: <20250428233136.37007-1-romain.beauxis@gmail.com> <20250428233136.37007-3-romain.beauxis@gmail.com> <20250429213511.GR4991@pb2> MIME-Version: 1.0 In-Reply-To: X-GND-State: clean X-GND-Score: -70 X-GND-Cause: gggruggvucftvghtrhhoucdtuddrgeefvddrtddtgddvieehvdegucetufdoteggodetrfdotffvucfrrhhofhhilhgvmecuifetpfffkfdpucggtfgfnhhsuhgsshgtrhhisggvnecuuegrihhlohhuthemuceftddunecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenfghrlhcuvffnffculdeftddmnecujfgurhepfffhvffukfhfgggtuggjsehgtderredttddunecuhfhrohhmpefoihgthhgrvghlucfpihgvuggvrhhmrgihvghruceomhhitghhrggvlhesnhhivgguvghrmhgrhigvrhdrtggtqeenucggtffrrghtthgvrhhnpedutedvhfduuedugedufefghefhvedvgffgffekhfdvgfdvtefftdejkeehteefheenucfkphepgedurdeiiedrieejrdduudefnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehinhgvthepgedurdeiiedrieejrdduudefpdhhvghloheplhhotggrlhhhohhsthdpmhgrihhlfhhrohhmpehmihgthhgrvghlsehnihgvuggvrhhmrgihvghrrdgttgdpnhgspghrtghpthhtohepuddprhgtphhtthhopehffhhmphgvghdquggvvhgvlhesfhhfmhhpvghgrdhorhhg X-GND-Sasl: michael@niedermayer.cc Subject: Re: [FFmpeg-devel] [PATCH v2 2/2] ogg/{vorbis, flac, opus}: Remove header packets from subsequent ogg streams from the demuxer output. X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Type: multipart/mixed; boundary="===============2159916820378250804==" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: --===============2159916820378250804== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="6U7kWbbXz06SusSc" Content-Disposition: inline --6U7kWbbXz06SusSc Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi Romain On Tue, Apr 29, 2025 at 05:42:22PM -0500, Romain Beauxis wrote: > Le mar. 29 avr. 2025 =E0 16:35, Michael Niedermayer > a =E9crit : > > > > Hi > > > > On Mon, Apr 28, 2025 at 06:31:36PM -0500, Romain Beauxis wrote: > > > --- > > > libavformat/oggdec.c | 26 ++++++++++---------- > > > libavformat/oggdec.h | 6 +++++ > > > libavformat/oggparseflac.c | 28 ++++++++++++++++++++= -- > > > libavformat/oggparseopus.c | 12 ++++++++++ > > > libavformat/oggparsevorbis.c | 12 ++++++++-- > > > tests/ref/fate/ogg-flac-chained-meta.txt | 2 -- > > > tests/ref/fate/ogg-opus-chained-meta.txt | 1 - > > > tests/ref/fate/ogg-vorbis-chained-meta.txt | 3 --- > > > 8 files changed, 68 insertions(+), 22 deletions(-) > > > > > > diff --git a/libavformat/oggdec.c b/libavformat/oggdec.c > > > index 5339fdd32c..5557eb4a14 100644 > > > --- a/libavformat/oggdec.c > > > +++ b/libavformat/oggdec.c > > > @@ -239,10 +239,6 @@ static int ogg_replace_stream(AVFormatContext *s= , uint32_t serial, char *magic, > > > os->start_trimming =3D 0; > > > os->end_trimming =3D 0; > > > > > > - /* Chained files have extradata as a new packet */ > > > - if (codec =3D=3D &ff_opus_codec) > > > - os->header =3D -1; > > > - > > > return i; > > > } > > > > > > @@ -605,20 +601,26 @@ static int ogg_packet(AVFormatContext *s, int *= sid, int *dstart, int *dsize, > > > } else { > > > os->pflags =3D 0; > > > os->pduration =3D 0; > > > + > > > + ret =3D 0; > > > if (os->codec && os->codec->packet) { > > > if ((ret =3D os->codec->packet(s, idx)) < 0) { > > > av_log(s, AV_LOG_ERROR, "Packet processing failed: %= s\n", av_err2str(ret)); > > > return ret; > > > } > > > } > > > - if (sid) > > > - *sid =3D idx; > > > - if (dstart) > > > - *dstart =3D os->pstart; > > > - if (dsize) > > > - *dsize =3D os->psize; > > > - if (fpos) > > > - *fpos =3D os->sync_pos; > > > + > > > + if (!ret) { > > > + if (sid) > > > + *sid =3D idx; > > > + if (dstart) > > > + *dstart =3D os->pstart; > > > + if (dsize) > > > + *dsize =3D os->psize; > > > + if (fpos) > > > + *fpos =3D os->sync_pos; > > > + } > > > + > > > os->pstart +=3D os->psize; > > > os->psize =3D 0; > > > if(os->pstart =3D=3D os->bufpos) > > > > > diff --git a/libavformat/oggdec.h b/libavformat/oggdec.h > > > index 43df23f4cb..09f698f99a 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 1 if the packet was a header from a chained bitstream. > > > + * 0 if the packet was a regular data packet. > > > + * -1 if an error occurred or for unsupported stream > > > + */ > > > int (*packet)(AVFormatContext *, int); > > > /** > > > * Translate a granule into a timestamp. > > > > ok, but seems unrelated >=20 > This is a new convention to allow the parser to know when to skip a > packet. Previous to that, return value of 1 did not have specific > meaning. Do you think this would merit a seperate patch ? I mean patch #1 changing the packet() return value and clearly stating that in the commit message and patch #2 using the new value ? I think it would make things clearer thx [...] --=20 Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The bravest are surely those who have the clearest vision of what is before them, glory and danger alike, and yet notwithstanding go out to meet it. -- Thucydides --6U7kWbbXz06SusSc Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iF0EABEKAB0WIQSf8hKLFH72cwut8TNhHseHBAsPqwUCaBFsIAAKCRBhHseHBAsP q/anAJ0YVZcUZQurMZO1RtjXcCpdR4UvPACcDEdG2wBLPa5MrLIwXWm7VHYEZQk= =JRGb -----END PGP SIGNATURE----- --6U7kWbbXz06SusSc-- --===============2159916820378250804== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". --===============2159916820378250804==--