From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: <ffmpeg-devel-bounces@ffmpeg.org> Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id 72F624D0AE for <ffmpegdev@gitmailbox.com>; Wed, 16 Apr 2025 01:21:57 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 5CE9F687DC1; Wed, 16 Apr 2025 04:21:53 +0300 (EEST) Received: from relay7-d.mail.gandi.net (relay7-d.mail.gandi.net [217.70.183.200]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id EE715687AAE for <ffmpeg-devel@ffmpeg.org>; Wed, 16 Apr 2025 04:21:46 +0300 (EEST) Received: by mail.gandi.net (Postfix) with ESMTPSA id 5387543217 for <ffmpeg-devel@ffmpeg.org>; Wed, 16 Apr 2025 01:21:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=niedermayer.cc; s=gm1; t=1744766506; 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=pHdE76aM1GpXNa/aBx56esJEqnwBIZGV1gWkBoK8Fow=; b=GNfJJzBXWRDwtLJz1DiucGQhRTHdGUee5qpwUQ5WLS5RdW5ufdtoKZm39o2nBCcPWEXisa 5RQJRwll3xKe4PFwPjfOawkzLSgKLXsMrviV/iufrtEJ6F6OgqlK6RXuvMobH2GEFn35QG tiYwdSGjNPipjy9Z/E+Vpv2crRpi89tRZgyu2lkxE9cpQBGjo5k7qrcoJnlQnZTejJDO9v SBSc80g2ztmlKBshZj2PIpBcu5DaPzNqP8naLyrL/8xOUZXibwQba8cinhJmX3IMMHGTeV MZ3i2EfdeaWQu0AXEjX6oxvyLSfjyi/HIvD/f9vfCYwKT8oVCdlJET8uLm0xgA== Date: Wed, 16 Apr 2025 03:21:45 +0200 From: Michael Niedermayer <michael@niedermayer.cc> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Message-ID: <20250416012145.GE4991@pb2> References: <20250411222719.2779176-1-michael@niedermayer.cc> <20250411222719.2779176-2-michael@niedermayer.cc> <DM8P223MB0365A90C10FBC9EF6512F9E1BAB12@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM> <20250414231950.GN4991@pb2> <DM8P223MB0365928500AB53069869DC99BAB32@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM> <20250415185538.GR4991@pb2> <DM8P223MB0365E10D38EF9174A32557DFBAB22@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM> <20250415225001.GW4991@pb2> <DM8P223MB0365EFE83DE448577271D852BAB22@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM> MIME-Version: 1.0 In-Reply-To: <DM8P223MB0365EFE83DE448577271D852BAB22@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM> X-GND-State: clean X-GND-Score: -85 X-GND-Cause: gggruggvucftvghtrhhoucdtuddrgeefvddrtddtgddvvdehtdegucetufdoteggodetrfdotffvucfrrhhofhhilhgvmecuifetpfffkfdpucggtfgfnhhsuhgsshgtrhhisggvnecuuegrihhlohhuthemuceftddunecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenfghrlhcuvffnffculdduhedmnecujfgurhepfffhvffukfhfgggtuggjsehgtderredttddvnecuhfhrohhmpefoihgthhgrvghlucfpihgvuggvrhhmrgihvghruceomhhitghhrggvlhesnhhivgguvghrmhgrhigvrhdrtggtqeenucggtffrrghtthgvrhhnpeeigeektdejudffjefhteegjedtgeettefggedthfejgfevhfetgeekjedtvdfhveenucfkphepgedurdeiiedrieejrdduudefnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehinhgvthepgedurdeiiedrieejrdduudefpdhhvghloheplhhotggrlhhhohhsthdpmhgrihhlfhhrohhmpehmihgthhgrvghlsehnihgvuggvrhhmrgihvghrrdgttgdpnhgspghrtghpthhtohepuddprhgtphhtthhopehffhhmphgvghdquggvvhgvlhesfhhfmhhpvghgrdhorhhg X-GND-Sasl: michael@niedermayer.cc Subject: Re: [FFmpeg-devel] [PATCH 2/2] avformat/id3v2: Check that decode_str() did advance X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches <ffmpeg-devel.ffmpeg.org> List-Unsubscribe: <https://ffmpeg.org/mailman/options/ffmpeg-devel>, <mailto:ffmpeg-devel-request@ffmpeg.org?subject=unsubscribe> List-Archive: <https://ffmpeg.org/pipermail/ffmpeg-devel> List-Post: <mailto:ffmpeg-devel@ffmpeg.org> List-Help: <mailto:ffmpeg-devel-request@ffmpeg.org?subject=help> List-Subscribe: <https://ffmpeg.org/mailman/listinfo/ffmpeg-devel>, <mailto:ffmpeg-devel-request@ffmpeg.org?subject=subscribe> Reply-To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Content-Type: multipart/mixed; boundary="===============2287346833572289656==" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" <ffmpeg-devel-bounces@ffmpeg.org> Archived-At: <https://master.gitmailbox.com/ffmpegdev/20250416012145.GE4991@pb2/> List-Archive: <https://master.gitmailbox.com/ffmpegdev/> List-Post: <mailto:ffmpegdev@gitmailbox.com> --===============2287346833572289656== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="kF0gz2N3+Psj2+iC" Content-Disposition: inline --kF0gz2N3+Psj2+iC Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Apr 15, 2025 at 10:59:07PM +0000, softworkz . wrote: >=20 >=20 > > -----Original Message----- > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > > Michael Niedermayer > > Sent: Mittwoch, 16. April 2025 00:50 > > To: FFmpeg development discussions and patches <ffmpeg- > > devel@ffmpeg.org> > > Subject: Re: [FFmpeg-devel] [PATCH 2/2] avformat/id3v2: Check that > > decode_str() did advance > >=20 > > On Tue, Apr 15, 2025 at 07:59:00PM +0000, softworkz . wrote: > > [...] > >=20 > > > - The representation of multi-values - both, internally and when > > > outputting as probe data - is a de-facto standard > >=20 > > The external handling in formats is specified in the corresponing > > specifications. ";" is certainly not correct for formats which > > natively support multiple values per key. > >=20 > > Internally, if you have a data structure that represents multiple > > authors, you certainly do not set it to one author and a string > > with a bunch of semicolons seperating multiple authors > >=20 > > Title: "Smile ;)" > > Author "Smily Face ;)" > >=20 > > is not 2 Titles and not 2 Authors and software that cannot handle that > > should not be used as reference IMHO > >=20 > > That said, anything that works is fine with me, > >=20 > > But internally it will be better to use a representation that is > > universal, generic and simple, ";" may seem to be that but only > > as long as you do nothing with it and dont care about corner cases > >=20 > > Ill leave this ";" question to everyone else, i have a backlog > > of quite a few things i need to work on >=20 > This is not a great outcome, because "leaving everyone else" means=20 > nothing will happen. I have a release to do, I have contracts that i should be working on, as iam a few month behind some deadline, I have a backlog of fuzzer issues i wanted to look at. I have a backlog of other things And this ";" thing is close to the last thing i want to spend my time on ;) >=20 > At least revert the rcombs patch until there's a conclusion, because > it really makes things worse than better with regards to FFprobe=20 > output. I guess thats reasonable. I will revert it, and then stay out of this unless it reaches the TC (where I as member will participate in finding a solution) > This will cause deserialization errors for many people in the world > who are processing FFprobe data. As said, ffprobe should not produce troublesome output thx [...] --=20 Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Modern terrorism, a quick summary: Need oil, start war with country that has oil, kill hundread thousand in war. Let country fall into chaos, be surprised about raise of fundamantalists. Drop more bombs, kill more people, be surprised about them taking revenge and drop even more bombs and strip your own citizens of their rights and freedoms. to be continued --kF0gz2N3+Psj2+iC Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iF0EABEKAB0WIQSf8hKLFH72cwut8TNhHseHBAsPqwUCZ/8GJQAKCRBhHseHBAsP qz8+AJ9Fj9Hjr6VUmsOBrg0AmUBIF1emWACfW/vr2A2XOlREc3fbotR2JTWrjEI= =Ywkt -----END PGP SIGNATURE----- --kF0gz2N3+Psj2+iC-- --===============2287346833572289656== 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". --===============2287346833572289656==--