From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.ffmpeg.org (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id 61A0A4C2B6 for ; Sun, 22 Jun 2025 22:11:00 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id 8751B68DEFD; Mon, 23 Jun 2025 01:10:55 +0300 (EEST) Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12olkn2100.outbound.protection.outlook.com [40.92.21.100]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id 137AD68DC9A for ; Mon, 23 Jun 2025 01:10:49 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=RWDo7IFsXhwfTseFU2VS2VsrvszC9pGs+SmoicWFUkzTxiReifjhXZUbnCpciWzYdgID/ZvlG85ouGU2dq6dnyotv+vNSLG3hxYMPeKLNiMpq67xznfVsK5FlqW/2d31Fm3EM06SIwljsucC5SSJ32JtyNhCCeDxM2Gg3ady3oZzaY1B2Xl/oisnzX3/arLMvCkYaUVfO9GsWmI3SR+qo3iiGkb4GeA/pMz8HOmCY8tEK6x1G61Sb3yX38+S8vK8rATt+EYZqrGgNRTA903LXNQqMIFsr5ixxp7mX0RIotD5DRESAg+wPIf/HmYFoXn1Nd1k0UOUegzGnu1NrgJeaA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=YYrE/b51XqDfJIndhSdlkL8aMWzoImy+EWF4Z76ixzg=; b=GX4Cv1dXMumWIba2jeDFJ6/uwcyWKAp3v4UGzT+3RoHdhlqwzj9bJHNx7bLULYfuqwrDzDknujPo2Q3e1ZnWL2OaN3t2mccez4Z4EpFgPi94avZAdE3bu4AJysm8MXsSw3y9AasMc/aiKT8c9O8+WOf3eLtpEThdH1+kQ2uFra5EwteZ9SAgx8R1x4cLibSBY7aLt68RPUyFBfDW2g3QfPJHFumNMx7G0qKK/MPc8qbwo9sKwqe8vJPmV1WrvAw5wizTYbqFFnLOeYICIHV3xnmgzeMqfTVtK13Xgi+ArzDRFCVnnhqDUxOlQg/6rAFa7V0tfoYzq563uCBWpbHpXA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=hotmail.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=YYrE/b51XqDfJIndhSdlkL8aMWzoImy+EWF4Z76ixzg=; b=Pr7QOhf//DP01O0FF5IA2O/+hnNVYjI0+itnZaQPdyoZsdwj+Yd49OE1FExASKibjnb2k294tmNGFhys78uU5YtKr6Op7eiDNzJXtiZQtc2KweYCOWaHN57VDwzHApgcFB97xnUsd+rf38qrlSQ1ztU8706OxoMPcYrJ/oR1UFDseBJuc1sv0JBG0RacYbWSioALn0tNtx+AgwtluxAIAzF0ZE8lzFpp49o/ktTJXB7yccG6zIbWiESO9TTRtXkYq7ndXvs8Zb8hzw75jIa6UIZPATZ2OadEPtsK81IngTjovYVj7p+YgeGCpjq+PuP+8WqPw/bq26ks+UfHgQGgWg== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by LV3P223MB1144.NAMP223.PROD.OUTLOOK.COM (2603:10b6:408:1db::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8857.27; Sun, 22 Jun 2025 22:10:45 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::bf09:8e9:b07f:98a7]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::bf09:8e9:b07f:98a7%6]) with mapi id 15.20.8857.026; Sun, 22 Jun 2025 22:10:45 +0000 From: "softworkz ." To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH 2/2] fftools/textformat: renamings in print_unit_int for consistency (cosmetic) Thread-Index: AQHb4dPPUNH5EtswOEKG1YVbjG3zm7QPv8Kg Date: Sun, 22 Jun 2025 22:10:45 +0000 Message-ID: References: <20250620110853.1448602-1-nicolas.gaullier@cji.paris> <20250620110853.1448602-2-nicolas.gaullier@cji.paris> In-Reply-To: <20250620110853.1448602-2-nicolas.gaullier@cji.paris> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-publictraffictype: Email x-ms-traffictypediagnostic: DM8P223MB0365:EE_|LV3P223MB1144:EE_ x-ms-office365-filtering-correlation-id: a3621d0b-c4b7-472b-eef4-08ddb1d9a2db x-microsoft-antispam: BCL:0; ARA:14566002|461199028|19110799006|8062599006|15080799009|8060799009|7092599006|40105399003|440099028|3412199025|102099032; x-microsoft-antispam-message-info: =?us-ascii?Q?7y0FFeSRQ97B/8pVRvYb1aMg7Bqp8kFzj+GonzusTyf4eyWTJLOk0YRTYuZX?= =?us-ascii?Q?PqprkJ0N+BXKWN5lvRVjuYZqYso7maPWkh8aElDF7hjA3nChHMf0yR204tSc?= =?us-ascii?Q?8//QSBaD3/rESVuuTE60f+ES9glbY2O5nHGNnyrdmDSwkNHGitKhXT9aA5IP?= =?us-ascii?Q?qY/tsv2ThPiwH+DWqv5atkx9OhMtkdAkDoWwawDYzhbSi5ElJW14ETqSET9L?= =?us-ascii?Q?/fAzyefExEyPtzMeEShPgpdomFI6l3yMWLhyqmbL/Gr0+QH1ttvTHCDradOx?= =?us-ascii?Q?DAhNME6UQOsvecrnnE1t8AfgfAL2lM4mY8cvN9qt5SEhq7uW4HaBAT7HycZw?= =?us-ascii?Q?OLyWrJBlnTnyuEYC1YTdVscFIND/nXEhxUhs68zss2PHaQO83Fh8DrAY4wn1?= =?us-ascii?Q?a2AcaerkhpZFzrjyWRJJ816eSAJUAf8Z0HjSWwg3MuWBieRWZivAv470RFWc?= =?us-ascii?Q?Z6MrfylIwG4zFvdKjLgqpLSY5bxitW2UQQ9dSsBRI7XrIWXL8kYPnWmXQ4C/?= =?us-ascii?Q?fgqAzNhMMSZ+xuQEKi1/KuW3aj8RhSvV5Tkmzbt3SX2+WqgJ/5ax81B6pxX0?= =?us-ascii?Q?FIbWJLG+vzWL6QOwZA7gIzs72iuY5zq+1KU7q4duBsSwiuwBYS6ky1+L0Xv5?= =?us-ascii?Q?aLPtCh5GTTKrOEqSlGSk1RhGF+KKMikyO1S6RypHBi6nfTVO+bB2+JsJSe/8?= =?us-ascii?Q?mpKE8wQHnF07DNMEFJ0M+vY90sh6iK78nax2FVbQm+yFv1WdIe8IXoEN1L4t?= =?us-ascii?Q?RccRGaplIiecilvRa1qDwvKTm0heQQ0y9B7Jd8wvwHsGJztCiUIGXQUbvw78?= =?us-ascii?Q?+qwFR5Wp/YVTGF+bmCY73v7bWOOFQ6XwKgsR62FhOapSg90l7GngrSeG2DtS?= =?us-ascii?Q?FBYUwf+8GZdqKoO16kqHy0EBVDuW6C2N+NxEB2khzct8VuAQqvGdXwTDQ1Io?= =?us-ascii?Q?a9nszObfuezWdqS9pSgy57trObwjkAUKZMDCgBLV4a/g48LWRmlWhMJM1qa5?= =?us-ascii?Q?ehwkyr8IvBMNkJ4TH2UgsSBWMxVBv/sSD7YT3+ZPIC/lVJaOnnN4Y7YHhsJ8?= =?us-ascii?Q?HcQAm7tVIStOoBQFAXQDxrx5mfMWX6DtHWQcLQ3jVLM35NS1sDRaR48qGWWo?= =?us-ascii?Q?KF9Rtg4ukGWUqvvrCJ5/wYCIB9rcx6Xv5Q=3D=3D?= x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?irKAWFxSk+PGPm+mMOS94Kt5Sw9+titgkmq0+6Sw2FF23iTH+Rd8e0+QEjow?= =?us-ascii?Q?UY9I32Qv7hTgNWXb3n3GoYHxquX7gI1F2KILr0KPUGIzgU/uJr+906TLjmsM?= =?us-ascii?Q?2jL+gNnnYACmzRiyArj9irRX8/VTFMm6D4D7C4N9Wsxxa80GikPhYuV28gf5?= =?us-ascii?Q?O+5uPQmUvZPC+OJKRRNUjeBEySl0lleEvM51sfF2rey1z3M+UY3Sl77P7p3t?= =?us-ascii?Q?y1meLZmV9ewCu4WBLKSuXuAY11pcA3LbXnUjYMZEMuTZDmHUsnrV/xDm2EKp?= =?us-ascii?Q?RM0kRIpd76YwaxvYlHjyehetIeZJzuKQq941SoryYYcAmfaj2xgDwgggCC4h?= =?us-ascii?Q?8x8dylQ3fVC67dig2VCV04xTaLNoROlAH/PvIDOfprsxJcw86bi3qCRRYIbN?= =?us-ascii?Q?7jQPUiR7xVbA3BvW+8SjJFIasW6t7BGIO3ehJcaEc6vwQRutkQQBPY/lCocR?= =?us-ascii?Q?ToAHfZ8iV41iD4ZO0F/mGXXa8UPg8c4WwDuUhqugufAWA6nho+Muh/LTmmzZ?= =?us-ascii?Q?hjvK6TFaL52+orCUF3KbIl9UPVX+q1dGoOnsusNEcM2Lnlby7gs9OIpqkPm6?= =?us-ascii?Q?J04Azcxl6D+YFoh8cOCbfdPC6slYVoV6ZnPKUahysOpGN9e0mvrX1JL68lxz?= =?us-ascii?Q?rsgDBcP6LL9eC2E0klNy8aP9joKbD2cPAPN/fTWLif/Q7J4kzww09qN4ZTbt?= =?us-ascii?Q?yu5WXZdF8y4K/UZeIJ9UvxoGomybn6qVxV5TS4Zqbr9btzW7T0WhwpoYgGXR?= =?us-ascii?Q?rKs6qurdtgU5OpogT5YVwK5ka4RE5SjwxVfIUiuHF4UpQ1GBFyLJXNsI9nD5?= =?us-ascii?Q?qskjvdy2OPjsahr0HV46PPYkcN6mAKLw9ygmEtWxfiiGkaoRJpT5tsE/3Az4?= =?us-ascii?Q?uYhnrUpSvULYXBIo0OEcvK9HpJokL05tJjN//oIxBbVQn9rNVVHxEyxwot5K?= =?us-ascii?Q?LX2WI3sXsck3Rq/PdjG9yjADFBccIERmEh4J+vIXpfZIc9P4w4bPd2dqXsHr?= =?us-ascii?Q?5NC74TMQGi+7i3rhH4licbELX1BxzT19JQtzkZ80Syw+2Q1ZK0uXMnMg853F?= =?us-ascii?Q?ej6a+sIfzgrcpYwwBB7/7QdLQi9xheT8o577Q2asOdOU6RirJtUuVm1AGsGe?= =?us-ascii?Q?UhplACowXNRXuF70GsWJBUhSUsOyzej0uif9M/jN38VrSgRmhWRXRrcSPd8q?= =?us-ascii?Q?3P88SHZEnh6bD86pk0CLVVUaLCbO6NqLmfYL/3ir3jutgiguVDkDZiEsLaY?= =?us-ascii?Q?=3D?= MIME-Version: 1.0 X-OriginatorOrg: sct-15-20-8534-20-msonline-outlook-c7cf3.templateTenant X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-Network-Message-Id: a3621d0b-c4b7-472b-eef4-08ddb1d9a2db X-MS-Exchange-CrossTenant-originalarrivaltime: 22 Jun 2025 22:10:45.2599 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-rms-persistedconsumerorg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: LV3P223MB1144 Subject: Re: [FFmpeg-devel] [PATCH 2/2] fftools/textformat: renamings in print_unit_int for consistency (cosmetic) 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 Cc: Nicolas Gaullier 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: > -----Original Message----- > From: ffmpeg-devel On Behalf Of > Nicolas Gaullier > Sent: Freitag, 20. Juni 2025 13:08 > To: ffmpeg-devel@ffmpeg.org > Cc: Nicolas Gaullier > Subject: [FFmpeg-devel] [PATCH 2/2] fftools/textformat: renamings > in print_unit_int for consistency (cosmetic) > > Signed-off-by: Nicolas Gaullier > --- > fftools/ffprobe.c | 2 +- > fftools/graph/graphprint.c | 2 +- > fftools/textformat/avtextformat.c | 4 ++-- > fftools/textformat/avtextformat.h | 2 +- > 4 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/fftools/ffprobe.c b/fftools/ffprobe.c > index 1346ed33c5..e8cde01407 100644 > --- a/fftools/ffprobe.c > +++ b/fftools/ffprobe.c > @@ -429,7 +429,7 @@ static void log_callback(void *ptr, int level, > const char *fmt, va_list vl) > #define print_ts(k, v) avtext_print_ts(tfc, k, v, 0) > #define print_duration_time(k, v, tb) avtext_print_time(tfc, k, v, > tb, 1) > #define print_duration_ts(k, v) avtext_print_ts(tfc, k, v, > 1) > -#define print_val(k, v, u) avtext_print_unit_int(tfc, k, v, > u) > +#define print_val(k, v, u) avtext_print_unit_integer(tfc, k, > v, u) > > #define REALLOCZ_ARRAY_STREAM(ptr, cur_n, new_n) > \ > { > \ > diff --git a/fftools/graph/graphprint.c > b/fftools/graph/graphprint.c > index e4c6886cf8..242eaf8ba1 100644 > --- a/fftools/graph/graphprint.c > +++ b/fftools/graph/graphprint.c > @@ -140,7 +140,7 @@ typedef struct GraphPrintContext { > #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, > gpc->opt_flags) > -#define print_val(k, v, u) avtext_print_unit_int(tfc, k, v, > u) > +#define print_val(k, v, u) avtext_print_unit_integer(tfc, k, > v, u) > > #define print_fmt(k, f, ...) do { \ > av_bprint_clear(&gpc->pbuf); \ > diff --git a/fftools/textformat/avtextformat.c > b/fftools/textformat/avtextformat.c > index 14bad9022b..396fe2f853 100644 > --- a/fftools/textformat/avtextformat.c > +++ b/fftools/textformat/avtextformat.c > @@ -437,11 +437,11 @@ static char *value_string(const > AVTextFormatContext *tctx, char *buf, int buf_si > } > > > -void avtext_print_unit_int(AVTextFormatContext *tctx, const char > *key, int64_t value, const char *unit) > +void avtext_print_unit_integer(AVTextFormatContext *tctx, const > char *key, int64_t val, const char *unit) > { > char val_str[128]; > struct unit_value uv; > - uv.val.i = value; > + uv.val.i = val; > uv.unit = unit; > avtext_print_string(tctx, key, value_string(tctx, val_str, > sizeof(val_str), uv), 0); > } > diff --git a/fftools/textformat/avtextformat.h > b/fftools/textformat/avtextformat.h > index 8316829af5..d9c14069eb 100644 > --- a/fftools/textformat/avtextformat.h > +++ b/fftools/textformat/avtextformat.h > @@ -169,7 +169,7 @@ void avtext_print_integer(AVTextFormatContext > *tctx, const char *key, int64_t va > > int avtext_print_string(AVTextFormatContext *tctx, const char > *key, const char *val, int flags); > > -void avtext_print_unit_int(AVTextFormatContext *tctx, const char > *key, int64_t value, const char *unit); > +void avtext_print_unit_integer(AVTextFormatContext *tctx, const > char *key, int64_t val, const char *unit); > > void avtext_print_rational(AVTextFormatContext *tctx, const char > *key, AVRational q, char sep); > > -- Hi Nicolas, thanks for the patch. Makes sense and LGTM. softworkz _______________________________________________ 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".