From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.ffmpeg.org (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id 7FE1C4AED9 for ; Wed, 23 Jul 2025 13:56:49 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id 1F1D668D8E8; Wed, 23 Jul 2025 16:56:37 +0300 (EEST) Received: from haasn.dev (haasn.dev [78.46.187.166]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id EB67C68D749 for ; Wed, 23 Jul 2025 16:56:27 +0300 (EEST) Received: from haasn.dev (unknown [10.30.1.1]) by haasn.dev (Postfix) with UTF8SMTP id 6BB8146D74 for ; Wed, 23 Jul 2025 15:56:27 +0200 (CEST) From: Niklas Haas To: ffmpeg-devel@ffmpeg.org Date: Wed, 23 Jul 2025 15:47:03 +0200 Message-ID: <20250723135626.1390296-1-ffmpeg@haasn.xyz> X-Mailer: git-send-email 2.50.1 MIME-Version: 1.0 Subject: [FFmpeg-devel] (no subject) 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: Changes since v1: - Correctly implement alpha mode tagging for JPEG XL - Set correct alpha mode for OpenEXR (which is always premultiplied) - Ensure -alpha_mode specified on the command line correctly propagates - Print out a warning when overriding the alpha mode explicitly Includes an unrelated fix for -chroma_sample_location also being ignored, since the fix for that was adjacent to the fix for the alpha mode field. Submitting this for re-evaluation. I decided in the end not to include full negotiation for the alpha mode, primarily because there is no filter currently existing that only handles one or the other. The only thing that represents a source of incompatibility is the codec itself, but that already errors out with a clearly defined error message when requesting an incompatible alpha mode - and I think the explicit fix (i.e. user inserts the corresponding (un)premultiply filter) is bettter than an implicit conversion, due to the potential for loss of information as well as breaking workflows that rely on additive blending (which only works for premultiplied alpha and thus cannot be converted - even explicitly). If the error message is really bothersome, we could add a small routine inside libavcodec for applying the unpremultiplication directly, but this would IMO be a layering violation. (Not to mention duplicating code) _______________________________________________ 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".