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 ESMTP id 76E2A46E1B for ; Thu, 13 Jul 2023 23:11:16 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id AF0E268C51E; Fri, 14 Jul 2023 02:11:14 +0300 (EEST) Received: from relay1-d.mail.gandi.net (relay1-d.mail.gandi.net [217.70.183.193]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id BC1FD68C26F for ; Fri, 14 Jul 2023 02:11:08 +0300 (EEST) Received: by mail.gandi.net (Postfix) with ESMTPSA id 1BE2F240003 for ; Thu, 13 Jul 2023 23:11:07 +0000 (UTC) Date: Fri, 14 Jul 2023 01:11:07 +0200 From: Michael Niedermayer To: FFmpeg development discussions and patches Message-ID: <20230713231107.GK1093384@pb2> References: <20230713105553.21052-1-anton@khirnov.net> <20230713105553.21052-25-anton@khirnov.net> MIME-Version: 1.0 In-Reply-To: <20230713105553.21052-25-anton@khirnov.net> X-GND-Sasl: michael@niedermayer.cc Subject: Re: [FFmpeg-devel] [PATCH 25/33] fftools/ffmpeg_filter: stop disregarding user-specified pixel format 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="===============2252503451249506329==" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: --===============2252503451249506329== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="LR9UEBmNK407lwl9" Content-Disposition: inline --LR9UEBmNK407lwl9 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Jul 13, 2023 at 12:55:45PM +0200, Anton Khirnov wrote: > When the user explicitly specifies a pixel format that is not supported > by the encoder, ffmpeg CLI will currently use some heuristics to pick > another supported format. This is wrong and the correct action here is > to fail. >=20 > Surprisingly, a number of FATE tests are affected by this and actually > use a different pixel format than is specified in the makefiles. > --- > doc/ffmpeg.texi | 3 +- > fftools/ffmpeg_filter.c | 35 +------------------ > tests/fate/fits.mak | 6 ++-- > tests/fate/lavf-video.mak | 2 +- > tests/fate/vcodec.mak | 4 +-- > .../{fitsdec-gbrap16le =3D> fitsdec-gbrap16be} | 4 +-- > .../fate/{fitsdec-gbrp16 =3D> fitsdec-gbrp16be} | 4 +-- > tests/ref/lavf/gif | 2 +- > 8 files changed, 13 insertions(+), 47 deletions(-) > rename tests/ref/fate/{fitsdec-gbrap16le =3D> fitsdec-gbrap16be} (79%) > rename tests/ref/fate/{fitsdec-gbrp16 =3D> fitsdec-gbrp16be} (79%) >=20 > diff --git a/doc/ffmpeg.texi b/doc/ffmpeg.texi > index 6769f8d305..08b11097b7 100644 > --- a/doc/ffmpeg.texi > +++ b/doc/ffmpeg.texi > @@ -1014,8 +1014,7 @@ Disable autoscale at your own risk. > @item -pix_fmt[:@var{stream_specifier}] @var{format} (@emph{input/output= ,per-stream}) > Set pixel format. Use @code{-pix_fmts} to show all the supported > pixel formats. > -If the selected pixel format can not be selected, ffmpeg will print a > -warning and select the best pixel format supported by the encoder. > + > If @var{pix_fmt} is prefixed by a @code{+}, ffmpeg will exit with an err= or > if the requested pixel format can not be selected, and automatic convers= ions > inside filtergraphs are disabled. The commit message makes this sound like a bugfix, while really this is removing a documented feature. It also breaks some scripts (fate is an example as it requires changes) If the removial of that feature is intended, it should be argued somewhere that this feature is never usefull. To me as a lazy person it surely feels usefull to be able to ask for both "exactly rgb" as well as something close to rgb (like bgr or gbrp) without needing to know what each individual codec uses to return R,G,B thx [...] --=20 Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Never trust a computer, one day, it may think you are the virus. -- Compn --LR9UEBmNK407lwl9 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iF0EABEIAB0WIQSf8hKLFH72cwut8TNhHseHBAsPqwUCZLCEhgAKCRBhHseHBAsP q9s+AKCUsN8U8sw+XkL0jxnHsfzSVha+7gCghj4hYk6LdQSJp53bqhEn4LutOzc= =SZ+l -----END PGP SIGNATURE----- --LR9UEBmNK407lwl9-- --===============2252503451249506329== 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". --===============2252503451249506329==--