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 E85CB46B8C for ; Fri, 4 Aug 2023 17:55:51 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 7310C68C6AC; Fri, 4 Aug 2023 20:55:48 +0300 (EEST) Received: from a27-226.smtp-out.us-west-2.amazonses.com (unknown [54.240.27.226]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 3314468C662 for ; Fri, 4 Aug 2023 20:55:41 +0300 (EEST) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/simple; s=b265wme4dq2usj3mmjgu4ufiz53apdpe; d=cosmin.at; t=1691171739; h=Subject:From:To:Date:Mime-Version:Content-Type:Content-Transfer-Encoding:References:Message-Id; bh=uAqePosokjJMhllF0RaXk6X+WECopylZsGdUUlVsmZw=; b=Hg5JZhmv0PVrFWzPDhgFbYcEYdnhc5tK8HrhKHqgwbBE8CQbSu+yyoF2US142gqD iOrA3DjZ2yuyV1cMcXdB9yGqE37uVSE7ynjRaSG2tcizYAdX9o/VFubImojunCU1otP HdN9vKQLyC7YMYIY4IkmpReTdlhzcX0TVHdr4FFSg4D2bY0Bx+9jNgu3V/Ovy+TxQnN pJVlSWHMonPmXHX4y4HzHSE84xcRr8kUYRszNsGHBTo5LrVloyOjJhQ0FwKZbXzQNQC SOkkdV+Gmv/zjJdCrWuerSHk2gNkMuuFUqxoWxJ0btIx+UaOB/jIffym5AWTffs2oT5 7zIaC3sqgA== DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/simple; s=hsbnp7p3ensaochzwyq5wwmceodymuwv; d=amazonses.com; t=1691171739; h=Subject:From:To:Date:Mime-Version:Content-Type:Content-Transfer-Encoding:References:Message-Id:Feedback-ID; bh=uAqePosokjJMhllF0RaXk6X+WECopylZsGdUUlVsmZw=; b=C2gTq20rFImx5RFCp/EFtpsGu/tMvUJNJaZ5S3VSuJoT3g5/dv71hg+39slsDmId /n47BR6WHj52LAeUgIszL7yuA00uwwLxPda6YX3KeUBOboImObDegpsQZ02piFT7amJ Pg3/Ux/qUmnLXKIJjn6CkPDnmYSsMrjMDxInX4ww= From: Cosmin Stejerean To: FFmpeg development discussions and patches Date: Fri, 4 Aug 2023 17:55:39 +0000 Mime-Version: 1.0 References: <870DEC0E-C94E-4432-88CB-E0E4F6A7162E@cosmin.at> X-Priority: 3 (Normal) X-Mailer: Amazon WorkMail Thread-Index: AQHZxvzgo/GZGv/YRNGdlYIZrHNHeg== Thread-Topic: [RFC][PATCH] libwebp: combine libwebp_anim and libwebp encoders into one X-Original-Mailer: Apple Mail (2.3731.700.6) X-Wm-Sent-Timestamp: 1691171738 Message-ID: <01010189c1b0d771-8340f30c-53d7-443f-98ee-14c6aec4c981-000000@us-west-2.amazonses.com> Feedback-ID: 1.us-west-2.An468LAV0jCjQDrDLvlZjeAthld7qrhZr+vow8irkvU=:AmazonSES X-SES-Outgoing: 2023.08.04-54.240.27.226 Subject: Re: [FFmpeg-devel] [RFC][PATCH] libwebp: combine libwebp_anim and libwebp encoders into one 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: On May 24, 2015 at 04:49PM, Michael Niedermayer wrote: > On Sun, May 24, 2015 at 06:39:05PM -0300, James Almer wrote: > > On 24/05/15 5:59 PM, Michael Niedermayer wrote: > > > On Sun, May 24, 2015 at 04:22:42PM -0300, James Almer wrote: > > >> Use either the WebPEncoder or WebPAnimEncoder APIs depending on availability > > >> of the latter. > > >> > > >> Signed-off-by: James Almer > > >> --- > > >> This is an RFC because the resulting encoder will use one of the two APIs, which > > >> is a change from the current behavior of having one encoder for each API. > > >> The new encoder was added only two days ago so removing it shouldn't be an issue. > > >> Basically, is there any benefit from using the native lavf muxer over letting > > >> WebPAnimEncoder do the entire process to justify having two encoders? > > >> > > >> The resulting ifdeffery is minimal now that the webp muxer can act as a raw muxer > > >> cleanly, so the only concern is the above. We can deal with cosmetics later, but > > >> the functional change needs to be done asap least we want to deal with deprecation > > >> nonsense if the new encoder makes it into a release. > > > > > > They are 2 different encoders, the old is partly implemented in > > > FFmpeg, the new does all steps in an external lib > > > The 2 variants work differently > > > command lines for the old encoder do NOT work with WebPAnimEncoder, > > > at least not with the current implementation we have > > > > My intention was to keep it all as a single encoder, using one of the two APIs depending > > on availability. If the AVOptions have different results on them then i guess it's not > > really feasible after all. So patch dropped. > > I dont know if its (im)possible with the new API to support the > stuff, just that it is not supported with the current > wraper/implementation we have There are still two encoders for animated webp, libwebp and libwebp_anim. Do we still need both? There was an attempt to combine these back in 2015 or so that was dropped at the time. At this point though WebPAnimEncoder has been around for years. Using libwebp_anim seems to be about 3x faster in my testing and produces smaller file sizes, and is selected by ffmpeg by default. New users may not realize that specifying `-c:v libwebp` is the inferior choice. Can we revisit merging these two? I propose dropping the current libwebp and renaming (or aliasing) libwebp_anim to libwebp and dropping support for libwebpmux < 0.4.0. I can submit a patch if there is agreement. - Cosmin _______________________________________________ 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".