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 E75424BBB1
	for <ffmpegdev@gitmailbox.com>; Sat,  3 May 2025 08:24:55 +0000 (UTC)
Received: from [127.0.1.1] (localhost [127.0.0.1])
	by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 250EA68B672;
	Sat,  3 May 2025 11:23:45 +0300 (EEST)
Received: from mail-pl1-f174.google.com (mail-pl1-f174.google.com
 [209.85.214.174])
 by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 5E83768B8C6
 for <ffmpeg-devel@ffmpeg.org>; Sat,  3 May 2025 11:23:43 +0300 (EEST)
Received: by mail-pl1-f174.google.com with SMTP id
 d9443c01a7336-22d95f0dda4so43145635ad.2
 for <ffmpeg-devel@ffmpeg.org>; Sat, 03 May 2025 01:23:43 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=gmail.com; s=20230601; t=1746260622; x=1746865422; darn=ffmpeg.org;
 h=cc:to:mime-version:content-transfer-encoding:fcc:subject:date
 :references:in-reply-to:message-id:from:from:to:cc:subject:date
 :message-id:reply-to;
 bh=WvXPrzt8PSNqxLdQ3SpO+LrpslrPHmFwpusVbiKR7II=;
 b=mwgURNtdlZ+fMNXFtdzE2XW4bKqxffhioo9yWUqhO9GaHtU6+pPi+ZoWUYl88qQPg1
 IBtGY4tdiJ940cKIHKCLETzLv+c1Z/s1HnHYUa+JuqVvBmNLXknGEQ5dy/AUrbfSp0ae
 G1AzddUMfypvdf16phi94pOoDMAD3kMSLLwGWYeqCMSPYAnU0H27eJLCYJkM3XsuwYQm
 kJxmK4CUc7KaPpG5XuHkWhhjpisUvHm8KxyCFq+8Oru8mJ12qFoKFEUbs03waPthlUmd
 g7fs2PiE2mwU5jWBDDCv1koTi7b1ALGqBx3j/JbRzlo0qVUm0OrsSNtB7Wwj9wk9GRZs
 QaCQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=1e100.net; s=20230601; t=1746260622; x=1746865422;
 h=cc:to:mime-version:content-transfer-encoding:fcc:subject:date
 :references:in-reply-to:message-id:from:x-gm-message-state:from:to
 :cc:subject:date:message-id:reply-to;
 bh=WvXPrzt8PSNqxLdQ3SpO+LrpslrPHmFwpusVbiKR7II=;
 b=J8pX2X4x8CYtnfAkoKNxEA+3SbV3tAG5/MrVwbobyIJ0cb04W5HYCLhgBjy5b+KOPs
 ZEhclU/p43G8dYooNteUlHx6jKi1eyukrr4K3MxzEsitQffN4N3Yvl4LlsTn+Je0NYn3
 Z6axhTCXTXnS9a6//7uktAaNJlAAJhYL5mLf00ChZiVhFYMQqqmurKivMC+eVL/vZDdC
 4oqkJpJ+HDUUdfyHU3+Y8jHttCCAaKuQXttK9feZuzuMfH26iPAvHYlJpo9XZceB7vez
 o7iK4G0Yqs7/A22FUI2PrBiYUdGhjYxMI2BExC/iwazRCUHINE/oMU2jFq0EwQ2lhIqV
 nnKw==
X-Gm-Message-State: AOJu0YxbvYog2o/1WmPRKyROLFPeolCMtbLHDnEPTTG9O4+MvFniXdK5
 XKrpBd/t/Z8Ws0HMkVfRJTAjxQ3Ns/rhDjI7Tvub/aV6soUiWpZ4weSNnA==
X-Gm-Gg: ASbGnct5EUYEG1aS6Ktrv1cdkDrwZoB5+MaqDrzNgk6SWyeHJbpInhws4isfZ+AQD0u
 MdKodX3NGAK0tXTeRED/zwWYxjzk8kWyh3NmArY0vceo/CGlwB/myWwJZbjZar1u+nIe3pLcU6N
 S13OuNn37obk9/eePbRyrk23zPY1KAFIhdTSvFRHOoee5I0TWr3G5Wo0tfOybIDxgWsonisZztV
 l+74KY7BFtmClB3uTkBYZLhiIy/TNcbGHZJku5DxTP9CJpwYnqtYBbbqN1F6QY9UV7BK7L8KxY5
 HLoX8YUWihwoYt3qXp2SPXBLcLjiveuee3TYcXCO9jc6Kd94YdQBCo8/M6M=
X-Google-Smtp-Source: AGHT+IH0LijBQKVGn2zaB6db+oYRAV63h+5nI3etqV0T6Z4Ib/1LVv3jkP4/wdxRl+HS3vdXObbxsQ==
X-Received: by 2002:a17:903:1110:b0:223:5ca8:5ecb with SMTP id
 d9443c01a7336-22e18c20b3cmr32446755ad.42.1746260621660; 
 Sat, 03 May 2025 01:23:41 -0700 (PDT)
Received: from [127.0.0.1] (master.gitmailbox.com. [34.83.118.50])
 by smtp.gmail.com with ESMTPSA id
 d9443c01a7336-22e151e93cbsm19715925ad.56.2025.05.03.01.23.41
 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128);
 Sat, 03 May 2025 01:23:41 -0700 (PDT)
From: softworkz <ffmpegagent@gmail.com>
X-Google-Original-From: softworkz <softworkz@hotmail.com>
Message-Id: <7d28b0f03c493d5cf49f6866ce6a02b0269da6d0.1746260566.git.ffmpegagent@gmail.com>
In-Reply-To: <pull.66.v9.ffstaging.FFmpeg.1746260565.ffmpegagent@gmail.com>
References: <pull.66.v8.ffstaging.FFmpeg.1745888404.ffmpegagent@gmail.com>
 <pull.66.v9.ffstaging.FFmpeg.1746260565.ffmpegagent@gmail.com>
Date: Sat, 03 May 2025 08:22:39 +0000
Fcc: Sent
MIME-Version: 1.0
To: ffmpeg-devel@ffmpeg.org
Subject: [FFmpeg-devel] [PATCH v9 09/15] fftools/textformat: Add flags param
 to function avtext_print_integer()
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/7d28b0f03c493d5cf49f6866ce6a02b0269da6d0.1746260566.git.ffmpegagent@gmail.com/>
List-Archive: <https://master.gitmailbox.com/ffmpegdev/>
List-Post: <mailto:ffmpegdev@gitmailbox.com>

From: softworkz <softworkz@hotmail.com>

Make this function work analog to avtext_print_string() which already
has a flags parameter.

Signed-off-by: softworkz <softworkz@hotmail.com>
---
 fftools/ffprobe.c                 |  2 +-
 fftools/textformat/avtextformat.c | 24 ++++++++++++++++++------
 fftools/textformat/avtextformat.h |  2 +-
 3 files changed, 20 insertions(+), 8 deletions(-)

diff --git a/fftools/ffprobe.c b/fftools/ffprobe.c
index 1277b1e4f9..444810cf43 100644
--- a/fftools/ffprobe.c
+++ b/fftools/ffprobe.c
@@ -422,7 +422,7 @@ static void log_callback(void *ptr, int level, const char *fmt, va_list vl)
     avtext_print_string(tfc, k, pbuf.str, 0);     \
 } while (0)
 
-#define print_int(k, v)         avtext_print_integer(tfc, k, v)
+#define print_int(k, v)         avtext_print_integer(tfc, k, v, 0)
 #define print_q(k, v, s)        avtext_print_rational(tfc, k, v, s)
 #define print_str(k, v)         avtext_print_string(tfc, k, v, 0)
 #define print_str_opt(k, v)     avtext_print_string(tfc, k, v, AV_TEXTFORMAT_PRINT_STRING_OPTIONAL)
diff --git a/fftools/textformat/avtextformat.c b/fftools/textformat/avtextformat.c
index 91469ef576..5b0f92d5cb 100644
--- a/fftools/textformat/avtextformat.c
+++ b/fftools/textformat/avtextformat.c
@@ -290,10 +290,20 @@ void avtext_print_section_footer(AVTextFormatContext *tctx)
     tctx->level--;
 }
 
-void avtext_print_integer(AVTextFormatContext *tctx, const char *key, int64_t val)
+void avtext_print_integer(AVTextFormatContext *tctx, const char *key, int64_t val, int flags)
 {
     const AVTextFormatSection *section;
 
+    av_assert0(tctx);
+
+    if (tctx->show_optional_fields == SHOW_OPTIONAL_FIELDS_NEVER)
+        return;
+
+    if (tctx->show_optional_fields == SHOW_OPTIONAL_FIELDS_AUTO
+        && (flags & AV_TEXTFORMAT_PRINT_STRING_OPTIONAL)
+        && !(tctx->formatter->flags & AV_TEXTFORMAT_FLAG_SUPPORTS_OPTIONAL_FIELDS))
+        return;
+
     av_assert0(key && tctx->level >= 0 && tctx->level < SECTION_MAX_NB_LEVELS);
 
     section = tctx->section[tctx->level];
@@ -445,10 +455,12 @@ int avtext_print_string(AVTextFormatContext *tctx, const char *key, const char *
 
     section = tctx->section[tctx->level];
 
-    if (tctx->show_optional_fields == SHOW_OPTIONAL_FIELDS_NEVER ||
-        (tctx->show_optional_fields == SHOW_OPTIONAL_FIELDS_AUTO
-            && (flags & AV_TEXTFORMAT_PRINT_STRING_OPTIONAL)
-            && !(tctx->formatter->flags & AV_TEXTFORMAT_FLAG_SUPPORTS_OPTIONAL_FIELDS)))
+    if (tctx->show_optional_fields == SHOW_OPTIONAL_FIELDS_NEVER)
+        return 0;
+
+    if (tctx->show_optional_fields == SHOW_OPTIONAL_FIELDS_AUTO
+        && (flags & AV_TEXTFORMAT_PRINT_STRING_OPTIONAL)
+        && !(tctx->formatter->flags & AV_TEXTFORMAT_FLAG_SUPPORTS_OPTIONAL_FIELDS))
         return 0;
 
     if (section->show_all_entries || av_dict_get(section->entries_to_show, key, NULL, 0)) {
@@ -504,7 +516,7 @@ void avtext_print_ts(AVTextFormatContext *tctx, const char *key, int64_t ts, int
     if ((!is_duration && ts == AV_NOPTS_VALUE) || (is_duration && ts == 0))
         avtext_print_string(tctx, key, "N/A", AV_TEXTFORMAT_PRINT_STRING_OPTIONAL);
     else
-        avtext_print_integer(tctx, key, ts);
+        avtext_print_integer(tctx, key, ts, 0);
 }
 
 void avtext_print_data(AVTextFormatContext *tctx, const char *key,
diff --git a/fftools/textformat/avtextformat.h b/fftools/textformat/avtextformat.h
index 87f57d8c24..8f406de322 100644
--- a/fftools/textformat/avtextformat.h
+++ b/fftools/textformat/avtextformat.h
@@ -138,7 +138,7 @@ void avtext_print_section_header(AVTextFormatContext *tctx, const void *data, in
 
 void avtext_print_section_footer(AVTextFormatContext *tctx);
 
-void avtext_print_integer(AVTextFormatContext *tctx, const char *key, int64_t val);
+void avtext_print_integer(AVTextFormatContext *tctx, const char *key, int64_t val, int flags);
 
 int avtext_print_string(AVTextFormatContext *tctx, const char *key, const char *val, int flags);
 
-- 
ffmpeg-codebot

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