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 CE71744647 for ; Tue, 20 Sep 2022 09:23:22 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 792A368BB2E; Tue, 20 Sep 2022 12:23:19 +0300 (EEST) Received: from mail0.khirnov.net (red.khirnov.net [176.97.15.12]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id BA42568BA08 for ; Tue, 20 Sep 2022 12:23:12 +0300 (EEST) Received: from localhost (localhost [IPv6:::1]) by mail0.khirnov.net (Postfix) with ESMTP id 59491240D03 for ; Tue, 20 Sep 2022 11:23:12 +0200 (CEST) Received: from mail0.khirnov.net ([IPv6:::1]) by localhost (mail0.khirnov.net [IPv6:::1]) (amavisd-new, port 10024) with ESMTP id gqddjsbMZR1j for ; Tue, 20 Sep 2022 11:23:08 +0200 (CEST) Received: from lain.khirnov.net (lain.khirnov.net [IPv6:2001:67c:1138:4306::3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "lain.khirnov.net", Issuer "smtp.khirnov.net SMTP CA" (verified OK)) by mail0.khirnov.net (Postfix) with ESMTPS id 00C6C240175 for ; Tue, 20 Sep 2022 11:23:07 +0200 (CEST) Received: by lain.khirnov.net (Postfix, from userid 1000) id 139151601B2; Tue, 20 Sep 2022 11:23:08 +0200 (CEST) From: Anton Khirnov To: FFmpeg development discussions and patches In-Reply-To: References: <20220919094604.4645-1-thilo.borgmann@mail.de> <166366300034.3205.15731879457980281823@lain.khirnov.net> Mail-Followup-To: FFmpeg development discussions and patches Date: Tue, 20 Sep 2022 11:23:08 +0200 Message-ID: <166366578804.5707.10237868598680678473@lain.khirnov.net> User-Agent: alot/0.8.1 MIME-Version: 1.0 Subject: Re: [FFmpeg-devel] [PATCH v4 0/6] Add display_matrix option 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: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: Quoting Nicolas George (2022-09-20 11:05:59) > Anton Khirnov (12022-09-20): > > I still see no convincing arguments in favor of adding all this > > complexity just so that we can stuff multiple options into one and get a > > WORSE user interface. > > > > Just add multiple options. > > We agree, except on the last point. > > All user-visible FFmpeg types should have a way for the user to input > them, self-contained and convenient with regard with the logic of the > type, and a function to print them in a way that can be parsed back. > > Channels layouts do not have multiple options, they have > av_channel_layout_from_string(). Rationals are not entered with > -framerate_num 24000 -framerate_den 1001, they are entered as a ratio, > and so on. It should be systematic. I have no problem with adding a -display_matrix option that allows expert users to specify all nine values of the display transformation matrix directly. But this set is not adding such an option, it is only handling rotation, flipping and scaling. Most users will think of these as three different operations and so it will be simplest - both for us and for our users - for them to be three separate options (with documentation stating the order in which they are applied). -- Anton Khirnov _______________________________________________ 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".