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