From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ffmpeg-devel-bounces@ffmpeg.org>
Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100])
	by master.gitmailbox.com (Postfix) with ESMTPS id 03DF34D9A4
	for <ffmpegdev@gitmailbox.com>; Mon, 21 Apr 2025 16:14:25 +0000 (UTC)
Received: from [127.0.1.1] (localhost [127.0.0.1])
	by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 0BE1A687BCD;
	Mon, 21 Apr 2025 19:14:19 +0300 (EEST)
Received: from mail-ed1-f47.google.com (mail-ed1-f47.google.com
 [209.85.208.47])
 by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 798E2687AA6
 for <ffmpeg-devel@ffmpeg.org>; Mon, 21 Apr 2025 19:14:12 +0300 (EEST)
Received: by mail-ed1-f47.google.com with SMTP id
 4fb4d7f45d1cf-5f624291db6so5033860a12.3
 for <ffmpeg-devel@ffmpeg.org>; Mon, 21 Apr 2025 09:14:12 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=gmail.com; s=20230601; t=1745252051; x=1745856851; darn=ffmpeg.org;
 h=user-agent:in-reply-to:content-disposition:mime-version:references
 :mail-followup-to:message-id:subject:cc:to:from:date:from:to:cc
 :subject:date:message-id:reply-to;
 bh=zV+gfhCbyR6nHMyWMUdklfyKKK84dHG5T+svYDEhl+s=;
 b=XhTxDBcb4mOTI5BWcJyb2GhrW5t0mVFDHQsDAShHhph6va8albR7q9tO3Nd9BHDIlX
 g5NLJoaqB42tDWZWvrCBz7olTZf4YwaWpT2p/TtgpPPG/kUylxwfyPAlbtmtmK+oV3cg
 tRpF3tk/V0NbRqDb26Amo/cpr7T0WDpqYIZ/3mmGAPKZ1uXLSM+02duxZt3yGVR6KlD3
 iovZEo4ILqhTQbG/bDVOXmA1vCYktF8g3UA+NWxZ/pB/T264sfDJZobnaTsmueTKXqgT
 yiArhOwJVt+jvYqgXJLlSBO+wWCgljoJ2TINTAjC5FmzXcgqeQ84D1hf12N3UqVpdE/K
 txyA==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=1e100.net; s=20230601; t=1745252051; x=1745856851;
 h=user-agent:in-reply-to:content-disposition:mime-version:references
 :mail-followup-to:message-id:subject:cc:to:from:date
 :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to;
 bh=zV+gfhCbyR6nHMyWMUdklfyKKK84dHG5T+svYDEhl+s=;
 b=fMllzKvGKhq/MG55lmnx0bMZc4nucPDFDJ0XbiSCFY8NOfvMlkrZH8fQr14nfp488H
 AMGUrJ+6C4131EDrlyk1o+ePJ8+jTj6oFKppPA0I3AU1XF9oShYzm98oTBOaflkWCX+c
 8wAd/E+yFSee5NvAcvW1bCrueFpnA+DwU8ivvEaO62ystwH8nRrYUVR/0D7HqyEpFFhR
 Dgcw9llxuZG79nokBEcpnSKzWxb2cPZfZOuD9A7TpQGfPnIXllY/jbHCOlzWb+OTi4CS
 AvMEAs8+PUHJQu6H8GW5FFNoRouI2stwYPPq0keYveA6BfFQM7lrse16OpXNaJmChTUw
 6brg==
X-Gm-Message-State: AOJu0YzmUeuMqGqVY6TRLLACbxj+WBymDy+LCPcafBcgjcYdTjrPpNCN
 nZULu5d3SrFwzLW/fsmgTdNTW8JWfcKEhw9vzwKSh2o/CbvPULctFJosow==
X-Gm-Gg: ASbGncvBLk2YhJRIplhyC6pFQACWPhWIgkSMJbTFBs+KxPh0hGo8kHEBUMQDVHVWfmh
 nhJ6fvC5A2KCDiKePbC6dAQt6zwz4lNzZoJVHlkX99jPDz5xYQg/OfXawRyaYTGkLPPyf3OafVq
 risR0QGfF2W+5EOUX/qMGTAiqqy6tE/w7VBWJR4WvuFFPFhC/PCCxXvIcDvqCr+y+9n6zMjpEzm
 CUrwbiq1sbM8dkHly6xAvIBI1g0V6dhDY0XABqlEmsQiFxXc3Aei8xMReuM7RYPQhEB3LOmEB4v
 hRczx5OP1tR4S/xRS4sHG5Pjd2f4
X-Google-Smtp-Source: AGHT+IFse37OmXnmjZMEg2FJjGRypiePKGjCdycMmGh+Sj+nVDzmi3FRpDVs2kp35TIUA5Ak46EIKA==
X-Received: by 2002:a05:6402:5249:b0:5e7:c438:83ec with SMTP id
 4fb4d7f45d1cf-5f62850bfadmr8956527a12.6.1745252051287; 
 Mon, 21 Apr 2025 09:14:11 -0700 (PDT)
Received: from mariano ([84.220.189.10]) by smtp.gmail.com with ESMTPSA id
 4fb4d7f45d1cf-5f6258342e5sm4688301a12.60.2025.04.21.09.14.10
 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);
 Mon, 21 Apr 2025 09:14:10 -0700 (PDT)
Received: by mariano (Postfix, from userid 1000)
 id 56942BFCE8; Mon, 21 Apr 2025 18:14:07 +0200 (CEST)
Date: Mon, 21 Apr 2025 18:14:07 +0200
From: Stefano Sabatini <stefasab@gmail.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Message-ID: <aAZuz57GMrjc3T5z@mariano>
Mail-Followup-To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>,
	softworkz <softworkz-at-hotmail.com@ffmpeg.org>
References: <pull.71.v2.ffstaging.FFmpeg.1745179839217.ffmpegagent@gmail.com>
 <pull.71.v3.ffstaging.FFmpeg.1745180141145.ffmpegagent@gmail.com>
MIME-Version: 1.0
Content-Disposition: inline
In-Reply-To: <pull.71.v3.ffstaging.FFmpeg.1745180141145.ffmpegagent@gmail.com>
User-Agent: Mutt/2.1.4 (2021-12-11)
Subject: Re: [FFmpeg-devel] [PATCH v3] fftools/ffprobe: Print size of
 attachments
X-BeenThere: ffmpeg-devel@ffmpeg.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: FFmpeg development discussions and patches <ffmpeg-devel.ffmpeg.org>
List-Unsubscribe: <https://ffmpeg.org/mailman/options/ffmpeg-devel>,
 <mailto:ffmpeg-devel-request@ffmpeg.org?subject=unsubscribe>
List-Archive: <https://ffmpeg.org/pipermail/ffmpeg-devel>
List-Post: <mailto:ffmpeg-devel@ffmpeg.org>
List-Help: <mailto:ffmpeg-devel-request@ffmpeg.org?subject=help>
List-Subscribe: <https://ffmpeg.org/mailman/listinfo/ffmpeg-devel>,
 <mailto:ffmpeg-devel-request@ffmpeg.org?subject=subscribe>
Reply-To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Cc: softworkz <softworkz@hotmail.com>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: ffmpeg-devel-bounces@ffmpeg.org
Sender: "ffmpeg-devel" <ffmpeg-devel-bounces@ffmpeg.org>
Archived-At: <https://master.gitmailbox.com/ffmpegdev/aAZuz57GMrjc3T5z@mariano/>
List-Archive: <https://master.gitmailbox.com/ffmpegdev/>
List-Post: <mailto:ffmpegdev@gitmailbox.com>

On date Sunday 2025-04-20 20:15:41 +0000, softworkz wrote:
> From: softworkz <softworkz@hotmail.com>
> 
> Signed-off-by: softworkz <softworkz@hotmail.com>
> ---
>     fftools/ffprobe: Print size of attachments
>     
>     
>     Versions
>     ========
>     
>     V2
>     
>     Fix spurious commit message text as noted by Andreas (thanks!)
>     
>     V3
>     
>     Also removed from the cover-letter part
> 
> Published-As: https://github.com/ffstaging/FFmpeg/releases/tag/pr-ffstaging-71%2Fsoftworkz%2Fsubmit_attachment_size-v3
> Fetch-It-Via: git fetch https://github.com/ffstaging/FFmpeg pr-ffstaging-71/softworkz/submit_attachment_size-v3
> Pull-Request: https://github.com/ffstaging/FFmpeg/pull/71
> 
> Range-diff vs v2:
> 
>  1:  dd35a56e25 = 1:  b3456eb9f6 fftools/ffprobe: Print size of attachments
> 
> 
>  doc/ffprobe.xsd   | 3 +++
>  fftools/ffprobe.c | 4 ++++
>  2 files changed, 7 insertions(+)
> 
> diff --git a/doc/ffprobe.xsd b/doc/ffprobe.xsd
> index b53b799227..8c2326e499 100644
> --- a/doc/ffprobe.xsd
> +++ b/doc/ffprobe.xsd
> @@ -319,6 +319,9 @@
>      <xsd:attribute name="nb_frames"        type="xsd:int"/>
>      <xsd:attribute name="nb_read_frames"   type="xsd:int"/>
>      <xsd:attribute name="nb_read_packets"  type="xsd:int"/>
> +
> +    <!-- attachment attributes -->
> +    <xsd:attribute name="attachment_size"  type="xsd:int"/>
>    </xsd:complexType>
>  
>    <xsd:complexType name="programType">
> diff --git a/fftools/ffprobe.c b/fftools/ffprobe.c
> index f5c83925b9..0da44b8c49 100644
> --- a/fftools/ffprobe.c
> +++ b/fftools/ffprobe.c
> @@ -1847,6 +1847,10 @@ static int show_stream(AVTextFormatContext *tfc, AVFormatContext *fmt_ctx, int s
>          else
>              print_str_opt("height",  "N/A");
>          break;
> +

> +    case AVMEDIA_TYPE_ATTACHMENT:
> +        if (par->extradata_size)
> +            print_int("attachment_size", par->extradata_size);

Would it make sense to print the extradata_size in each case to make
this more generic?

_______________________________________________
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".