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 01B9744A84 for ; Sun, 7 May 2023 16:59:44 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 228F768C070; Sun, 7 May 2023 19:59:42 +0300 (EEST) Received: from mail-qv1-f53.google.com (mail-qv1-f53.google.com [209.85.219.53]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 1C88B68A4F2 for ; Sun, 7 May 2023 19:59:34 +0300 (EEST) Received: by mail-qv1-f53.google.com with SMTP id 6a1803df08f44-61b79b97ed8so17548246d6.1 for ; Sun, 07 May 2023 09:59:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=obe-tv.20221208.gappssmtp.com; s=20221208; t=1683478772; x=1686070772; h=to:subject:message-id:date:from:in-reply-to:references:mime-version :from:to:cc:subject:date:message-id:reply-to; bh=yLdPhEQq5NPNX5+X0sv9W6mxru1RkqlmhMhIPb6lATM=; b=o6XG0bVz9rIpTgYh80jK0Y7pQdOF625HgLS8gthGCZXL9ords6Z7zvr4Qce0oIvJpu 7DuwCcw14/1BHI7uE1yaXSxp/qPen5VbR4mmLMMDGd6Yi4PgP+8H1crw5f7WQ6y1yldP ZCkfr3eHCAOUfvyCmHG2dGIrZU03V5JoTchXl1+6HhsKnBYDHmcv8RSTauedJXBOiA7x xhFoEAbluxhABoVydsYGt/EJ6BC2s2WOHI0a7BkjzPNUNbHrVcWWo5ghMuPreoIbklPv SMDhgjIJr57KocY1jD/CorWmt4VHHor3/belIKnp/YEdwJMvEO+be3IK15gJE7uz7I1V 8yng== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1683478772; x=1686070772; h=to:subject:message-id:date:from:in-reply-to:references:mime-version :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=yLdPhEQq5NPNX5+X0sv9W6mxru1RkqlmhMhIPb6lATM=; b=FG/V88jVfJkvZ/L30SifaDVdbIO3+ZLAI0XZcv3b0hFCIpD8c6kqDTu1qPvpJSQCmF MqMvoWdLZRoRkSusXdC9+aJoW6yZ4QD5YHdE7Bv2GwfAEJ96HiXy+6vAkBeIKBY09Xf8 lDPJkybLms5w+EUqjZhTKpnZ/t2UKaBEFv3XRpUg/B6mslKZ6xsvgoPT1NwyTkVrnpFw WO70jFpHTazIAmMDo49FKQSYGvgwFYxdXTf7SirleXzmywq/T4+CnoE/eqiwFjwccF+D J3KYS9NcHqZv6RLe1VtpawRd88MddQlnWVEwwyQKcAhw1TsZSV4fFeZpGKfve1HjgCEn ojHg== X-Gm-Message-State: AC+VfDzVC6F8sqHGDfSILsmgsdeUEzAHj7jaZiK7goEOkUhZ+KbGFhpH qVXENjcn71pa+19O6XrcMqW5WjM87/dgHNrBxEY0lBMQpeiZ+jSP X-Google-Smtp-Source: ACHHUZ7+4GKIf8xFTgdaqlF2PeJp9+Aqd/prrok0D4mmUtGQYCXsvc+gbxZTZHkiGvVCjWPOWPStNVrV4NbD2IYOzNI= X-Received: by 2002:a05:6214:519d:b0:615:a787:6d2a with SMTP id kl29-20020a056214519d00b00615a7876d2amr11951209qvb.3.1683478772360; Sun, 07 May 2023 09:59:32 -0700 (PDT) MIME-Version: 1.0 References: <20230507133255.20881-1-anton@khirnov.net> In-Reply-To: <20230507133255.20881-1-anton@khirnov.net> From: Kieran Kunhya Date: Sun, 7 May 2023 17:59:22 +0100 Message-ID: To: FFmpeg development discussions and patches X-Content-Filtered-By: Mailman/MimeDel 2.1.29 Subject: Re: [FFmpeg-devel] [PATCH 01/13] lavu/frame: extend AVFrame.repeat_pict documentation 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 Sun, 7 May 2023, 14:34 Anton Khirnov, wrote: > --- > libavutil/frame.h | 18 ++++++++++++++++-- > 1 file changed, 16 insertions(+), 2 deletions(-) > > diff --git a/libavutil/frame.h b/libavutil/frame.h > index f2b56beebb..ed3f199ce1 100644 > --- a/libavutil/frame.h > +++ b/libavutil/frame.h > @@ -491,8 +491,22 @@ typedef struct AVFrame { > void *opaque; > > /** > - * When decoding, this signals how much the picture must be delayed. > - * extra_delay = repeat_pict / (2*fps) > + * Number of fields in this frame which should be repeated, i.e. the > total > + * duration of this frame should be repeat_pict + 2 normal field > durations. > + * > + * For interlaced frames this field may be set to 1, which signals > that this > + * frame should be presented as 3 fields: beginning with the first > field (as > + * determined by AV_FRAME_FLAG_TOP_FIELD_FIRST being set or not), > followed > + * by the second field, and then the first field again. > + * > + * For progressive frames this field may be set to a multiple of 2, > which > + * signals that this frame's duration should be (repeat_pict + 2) / 2 > + * normal frame durations. This isn't correct, a progressive [coded] frame is allowed to have its first field repeated. There is a difference between the coded type and the display method of a frame. The documentation suggests otherwise. Kieran _______________________________________________ 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".