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 396C147C27 for ; Thu, 11 Apr 2024 20:18:10 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id AF05268C946; Thu, 11 Apr 2024 23:18:07 +0300 (EEST) Received: from mail-pj1-f54.google.com (mail-pj1-f54.google.com [209.85.216.54]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 89A4B68AA7E for ; Thu, 11 Apr 2024 23:18:05 +0300 (EEST) Received: by mail-pj1-f54.google.com with SMTP id 98e67ed59e1d1-2a5ef566c7aso122135a91.1 for ; Thu, 11 Apr 2024 13:18:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ob-encoder-com.20230601.gappssmtp.com; s=20230601; t=1712866682; x=1713471482; darn=ffmpeg.org; h=to:subject:message-id:date:from:in-reply-to:references:mime-version :from:to:cc:subject:date:message-id:reply-to; bh=6a6MVdEmn8r9S/YCGnh6sUg4qOUbrddCcvC8ibJusCI=; b=Ee0Dtlf8Uy08ivG0DTMGQFF6cMyoiCy52jijuDBu4x6cHiwy5iLwZ1RkL+d/UfpL3z UFh34e5et0tS4/ofnRlgj976hR+OuyCejIeHyOPULT1O0UQ+uUJAsc7bSWg7w43LU6N/ kBjnh8W10v0vphj3PciPkVchHY5Fi5qpTPBrS8rCmeiOKuhy/ncL8dZh6YotLjBUSn8Q AEOF91G0XKtSYtLNP4+DCj/TV+5co+1B2Pdz5pSjs20sSVWf9TefdBROhuE5dOnS6Xfl zs6V4DQnEX9NZb/+NVruqS0ejbv/qs1huHrTvlf9Y2MMbBzToFUzmSMrUsl+AR3kfMsP 9VXQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1712866682; x=1713471482; 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=6a6MVdEmn8r9S/YCGnh6sUg4qOUbrddCcvC8ibJusCI=; b=wqjT38F0DvKZ0oYI/r2Fd+xJtCH5XpWA0wIIp/6+VaIwGOwCnbOkKL2fGCCfDSdkH0 z/dqTQIpju9CZZ9DTDIn6wZR1mZzxdLdJ+yxQDKGlAWASyYlrMXLoR45xnn123uT1NUk SxCR9HSE77EtUMCQR/sXJke/oowCaGJeTAl5ge/DHq9d4s16oJQm2YhIamvzQYnJBnfR T3PvQ2JTPca0oa8KGctXluV8H1bpvu/2AiWaa29v/fXabbj1bFEzFZrOOpQbCGbp3vgy wVxeDwd6v9HpOWbEL4CUzL6422Y5GPnfJsb/IAORh9IPHgDXz7t7OYppczkI/cxPorqJ 3Hgg== X-Gm-Message-State: AOJu0Yw80nfuoIRD8b+PL3jTQlS4SdH6tiFDTfRhjw5Ejnzvqt93p5Ej 3jqb3mNKLtwofCbQNrUWQlWmyEkpITFIFLiTgGXTlS8oUqFl9W32Lx4Bf3kOGBsEpQLUu3gqIuA gGQjji048m97fvhkq1E7Sr3FubxI3k6ADntDsS7bK+u+BbrrpcQU= X-Google-Smtp-Source: AGHT+IHFqUx3RfHBchZYQam8l/vss0vdvL8pp3omP9wZLJP0HUWWdpOj76zaOnBFeD53JX34P2tsgyCtUX0kDcs5H0s= X-Received: by 2002:a17:90b:803:b0:2a6:43c:44c6 with SMTP id bk3-20020a17090b080300b002a6043c44c6mr703030pjb.29.1712866682481; Thu, 11 Apr 2024 13:18:02 -0700 (PDT) MIME-Version: 1.0 References: <20240411154223.61774-1-jamrial@gmail.com> In-Reply-To: <20240411154223.61774-1-jamrial@gmail.com> From: Kieran Kunhya Date: Thu, 11 Apr 2024 21:17:49 +0100 Message-ID: To: FFmpeg development discussions and patches X-Content-Filtered-By: Mailman/MimeDel 2.1.29 Subject: Re: [FFmpeg-devel] [PATCH] avutil/frame: flag A53_CC side data type as allowing multiple entries 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 Thu, 11 Apr 2024, 16:42 James Almer, wrote: > Some interlaced h264 samples may export one caption per field. > > Signed-off-by: James Almer > --- > libavutil/frame.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/frame.c b/libavutil/frame.c > index 0775e2abd9..fdac6f8ee2 100644 > --- a/libavutil/frame.c > +++ b/libavutil/frame.c > @@ -30,7 +30,6 @@ > > static const AVSideDataDescriptor sd_props[] = { > [AV_FRAME_DATA_PANSCAN] = { "AVPanScan" }, > - [AV_FRAME_DATA_A53_CC] = { "ATSC A53 Part 4 > Closed Captions" }, > [AV_FRAME_DATA_MATRIXENCODING] = { "AVMatrixEncoding" }, > [AV_FRAME_DATA_DOWNMIX_INFO] = { "Metadata relevant to > a downmix procedure" }, > [AV_FRAME_DATA_AFD] = { "Active format > description" }, > @@ -55,6 +54,7 @@ static const AVSideDataDescriptor sd_props[] = { > [AV_FRAME_DATA_AMBIENT_VIEWING_ENVIRONMENT] = { "Ambient viewing > environment", AV_SIDE_DATA_PROP_GLOBAL }, > [AV_FRAME_DATA_SPHERICAL] = { "Spherical Mapping", > AV_SIDE_DATA_PROP_GLOBAL }, > [AV_FRAME_DATA_ICC_PROFILE] = { "ICC profile", > AV_SIDE_DATA_PROP_GLOBAL }, > + [AV_FRAME_DATA_A53_CC] = { "ATSC A53 Part 4 > Closed Captions", AV_SIDE_DATA_PROP_MULTI }, > [AV_FRAME_DATA_SEI_UNREGISTERED] = { "H.26[45] User Data > Unregistered SEI message", AV_SIDE_DATA_PROP_MULTI }, > }; > > -- > 2.44.0 > But we merge the captions into a single side data element, no? 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".