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 48EDF40C14 for ; Wed, 5 Oct 2022 18:55:01 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 164ED68B9A5; Wed, 5 Oct 2022 21:54:59 +0300 (EEST) Received: from iq.passwd.hu (iq.passwd.hu [217.27.212.140]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id EE03568B6D3 for ; Wed, 5 Oct 2022 21:54:52 +0300 (EEST) Received: from localhost (localhost [127.0.0.1]) by iq.passwd.hu (Postfix) with ESMTP id DE64BE6FCC for ; Wed, 5 Oct 2022 20:54:50 +0200 (CEST) X-Virus-Scanned: amavisd-new at passwd.hu Received: from iq.passwd.hu ([127.0.0.1]) by localhost (iq.passwd.hu [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id EBgVmpn1-Ufq for ; Wed, 5 Oct 2022 20:54:46 +0200 (CEST) Received: from iq (iq [217.27.212.140]) by iq.passwd.hu (Postfix) with ESMTPS id 77DCCE444F for ; Wed, 5 Oct 2022 20:54:46 +0200 (CEST) Date: Wed, 5 Oct 2022 20:54:46 +0200 (CEST) From: Marton Balint To: FFmpeg development discussions and patches In-Reply-To: <166487620906.5794.5456148015950663376@lain.khirnov.net> Message-ID: References: <20220928093213.947-1-anton@khirnov.net> <5f92999c-2097-5016-daff-7ecddbfac1f3@passwd.hu> <166487620906.5794.5456148015950663376@lain.khirnov.net> MIME-Version: 1.0 Subject: Re: [FFmpeg-devel] [PATCH 1/3] lavc/encode: make sure frame timebase matches encoder, when set 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-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On Tue, 4 Oct 2022, Anton Khirnov wrote: > Quoting Marton Balint (2022-09-28 21:54:11) >> >> >> On Wed, 28 Sep 2022, Anton Khirnov wrote: >> >>> AVFrame.time_base has been added recently, but is currently not used for >>> anything. Prepare for its use in encoders by rejecting frames where >>> time_base is set, but differs from the AVCodecContext one. >> >> How is that not an API break? Users can encode AVFrames with anything in >> the AVFrame->time_base right now, if you change that behaviour, that will >> surely break some code. That is why it was explicitly documented that >> it will be ignored by encoders by default. > > Why would there be anything in that field? No code we have currently > sets that field or does anything with it. It is a public field which was explicitly documented to be ignored by filters or encoders. The user could store any data in it, because the documentation of the field ensured it will not be a problem. If you read back the old threads which added AVFrame->time_base you will find the reasoning behind the original comments, in fact, you suggested the actual wording for the documentation of the field, and now you want now to change the semantics of the field which contradicts the existing documentation... Usually we introduce a new field and deprecate the old if we want to do something like this. One could argue that this break is "small" enough, to not dance around it, but I don't really see the benefit of the change in the first place. So the real question is why do you want to start using AVFrame->time_base in encoders, and what is the feature which is undoable with the current AVCodecContext->time_base? Thanks, Marton > There is no valid reason for > the users to be setting it on the frames they send to lavc. > > As for "it would have worked before', there are many precedents where > some nonsensical parameter combination would "work", but then we'd add a > check and it would start returning errors. Callers should not be setting > random fields to random values and expect things to work. > > Would applying this patch after a major bump alleviate your concerns? We > wanted to have one for a few months already. > > -- > 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". > _______________________________________________ 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".