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 ESMTPS id D4FD44C599 for ; Fri, 7 Feb 2025 13:22:12 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id AB93468B792; Fri, 7 Feb 2025 15:22:08 +0200 (EET) Received: from EUR03-AM7-obe.outbound.protection.outlook.com (mail-am7eur03on2123.outbound.protection.outlook.com [40.107.105.123]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 8E39168BD59 for ; Fri, 7 Feb 2025 15:22:02 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=MQI4+cbgj7CEcFQ7M9Gdz0fk0aKMTtvE5J+rS/4Rn2BGtM/6qtZYcWkqC3rgFtec8hdp4mN5FPTGNVThjQJvy6iQGfk8QoTuEUsWomzJxowfi6oYv7FSt0v43QiNecSr1548mJ7d20aQpMmrpag4XVaoxgQmL2mvxU6I2VHr5oe8kYuAogcXBYs9lHZu4vIplZvfG++tc/SLZ/mkps1RdToWotsPM7crYHCc2eiC5ujM0Bp4D+PBdrHUAUqvCUY3FEUf8FYXCGq8S9/0gw5go4upwF/eUPujT5O2/nMpUl8ywyKIivm48Hkt2RMolDMUvIfwyFyAJdNMjC1vAhbjvw== 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=lgB4p6+5Sfni9e2n0nticXUtHEDa/kCO5qTSLGvXcDY=; b=F24gjf7t18qLql6A6BUmn+UxE24YAWmGZXpVbI1WfuqT/f5iHu/V3egicsJ867gkUyiKplWT6WEE2RGMvs4bGf1aK5U0NY8aw8PKuUW3GrtVoX9LixaSNozv2e145it6q8vOV2lc33C+yQaq8YNLeSj+rgZMDQVk3t+geDyMFWZfL0Jea+OrbNxaiz48HSg3Oj675tF9yhPjsDX2reZLMAWFL1E17quCv4TfcwdRn5VhBVinyB21Db7p73OQsSktUJ9kQswMIiL25gWyR/ANa1wl76/z48ZP9+VTFMrcdK6a4Gi88T2/LfvaFoYR6QiIGnjui70WcTRaxihg0dpKlA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=noa-archive.com; dmarc=pass action=none header.from=noa-archive.com; dkim=pass header.d=noa-archive.com; arc=none Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=noa-archive.com; Received: from DB4PR09MB5847.eurprd09.prod.outlook.com (2603:10a6:10:38d::8) by GV2PR09MB6325.eurprd09.prod.outlook.com (2603:10a6:150:d6::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8398.24; Fri, 7 Feb 2025 13:21:58 +0000 Received: from DB4PR09MB5847.eurprd09.prod.outlook.com ([fe80::ebcf:249e:da70:d69]) by DB4PR09MB5847.eurprd09.prod.outlook.com ([fe80::ebcf:249e:da70:d69%4]) with mapi id 15.20.8398.021; Fri, 7 Feb 2025 13:21:58 +0000 Message-ID: <6d7c527b-b834-4c08-9d16-90e033f28f45@noa-archive.com> Date: Fri, 7 Feb 2025 14:21:57 +0100 User-Agent: Mozilla Thunderbird To: ffmpeg-devel@ffmpeg.org References: <63ab3a941166604541ce91cc37137edef4cbac93.1738915048.git.ffmpegagent@gmail.com> <2bd0c238-546d-4cb2-8778-e9d6f658fa3a@noa-archive.com> Content-Language: en-US From: Tobias Rapp Organization: NOA GmbH In-Reply-To: X-ClientProxiedBy: FR4P281CA0269.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:e8::16) To DB4PR09MB5847.eurprd09.prod.outlook.com (2603:10a6:10:38d::8) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DB4PR09MB5847:EE_|GV2PR09MB6325:EE_ X-MS-Office365-Filtering-Correlation-Id: 05625ca0-a03e-42a6-13a9-08dd477a661a X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; ARA:13230040|376014|366016|1800799024|13003099007|8096899003|7053199007; X-Microsoft-Antispam-Message-Info: =?utf-8?B?TTdsc3ZhcmZsQkdZT2dVVzdpLzFMV0ZQNGdEUG95ekpTTXlIbUM5L1dtemhs?= =?utf-8?B?R2htd0ZsTkFPUDVPam0wM1g1OXpKaE9IdVpnZldYZWdrTmI4MFk1bWJwZ21H?= =?utf-8?B?TzhYTVM2SUZhLzBzT2Vabk92Z21abEtHV2Ywc0RSTnE3clo1VzJ5WExOK3JZ?= =?utf-8?B?Zm1keWswaEZ5TUtOemxqYXFvcjFrR1pNWmFjbDhMdnB3ZUJwZ1dsbUhqTjJa?= =?utf-8?B?NmltMm40WWtjRWhybkhjV05YZWlvNkJqQk5nbk9yd0pJcmZCcUpSYjFJTEtv?= =?utf-8?B?YmJWMnFpbm1zYUlMVWh2RHNWai9aVnBDZDIzZHVpQ0RpUEZlVit6RzkwMW81?= =?utf-8?B?WWE3ZjRyZThpWmlWdndtVytCSVM2ZTdPY2RnbWZ4aTMwdXZOUXp5cmY0S0Qr?= =?utf-8?B?Vk5tVUw3TUFwSzhmUWx1OWR3eTVyUFo3ZTkrM0h2aWlTc3NZb1RlTGlhSmta?= =?utf-8?B?UFpLOCtwOTFhbHZsVndtL1l1c0tzK3Nodld4eEQwZzVQbjE1a2VNc3F2ZkVk?= =?utf-8?B?TklTNlMvNEZOc241Q2lMa3ROcFJ2NjhVdDMwcWpGQ096VWFwZnE3S3lrWXVz?= =?utf-8?B?N21SM1R1RHJoc0Fhd2xBMnJvY3NXbnYyT0YvMll5cGsvSkxXVm9wRUp4NXFn?= =?utf-8?B?Z210NEp2VXhFLzc5aHlkNnlFMk1xb3kxVzZrRUhUc2c5WWtFR0lvbEhGdGN2?= =?utf-8?B?aXJIQ2ZrczJwYmJGZitURzJLYmZ4WTZ2K3I3amIwK3Z0TG0xU1A4cGpGSVJW?= =?utf-8?B?blpvSzJRL1RDenVFOHkwLzB1bW1wUG1ORjdoaFNJY0h1Mm0wd2VNTVpsVHJm?= =?utf-8?B?SmllK2dSaVVienRaTkpIUGFMMTJlL1IxOFU1UVYyb09oNFkxaWZIaVJhaG9P?= =?utf-8?B?c29xdWhtS2x2RDBFWFBRZUNYbGduMTB5Sk9pRE5FdzBVbWxqWWp0QkUwR2p6?= =?utf-8?B?SlNhdmZrOUhCRGZNMnpoQ1JhMXgwVjVrMk9DUUpJbEl3VGF2b0l5S0hCN21u?= =?utf-8?B?NmRBUjk4ZEwvNHFQMFB0bFZ1T0FMK2Y1RjJkMmhwNFNaQ21SVWgyNFBDUEIz?= =?utf-8?B?VmdYb2xDVXRoQSthSUY3VUYralpPc0t2cldZdHk1d1RTZ2MvdjZLOWxTUVd6?= =?utf-8?B?VnJ6S1hBWXFqWDNGK1RweG5naVJTTkZmN3JLTUNtdXBtZlN2NUtDa0xWeCtq?= =?utf-8?B?NlFrNVluK05EdytPZFRPK29qbmY5VFBvZWF4azhaYlVhNHk4YWdCU1ZQWUxP?= =?utf-8?B?OWVBZFlhWEFvdG1TcVRXbTRoRkRUTDFsV0xCRldCU1Y0cmNYMWtiK01TZXA4?= =?utf-8?B?MTdqcENmVEtkMG5tSFRFTkNZWGpKL2xlczJXWTdwdUhtSTNzTUdSdlROZTNm?= =?utf-8?B?WG9zTWI5c2VGUGhWSXpTcHd3QTF1cE1COFNNVlp2eWlKQkVkSFlLKyt4elpy?= =?utf-8?B?dzFYOGp5L1Q4WVZla2owTmNVbzg4bEF3Q29VbnFyck5UVFBNT3BYWW1vSFVl?= =?utf-8?B?UUFEOXNhNVZqYlFPcitTUFpQcEI2Zk9Zd2UxUXQyN2pDV2JPUWpmY3M5T3Nn?= =?utf-8?B?M2VRZkhzUDhDMW5aR29pcjB4OS82SEgwYW1vN1YwOXBQb05ZajFYcHVaQ3Vo?= =?utf-8?B?YmZGQk5Kdm9ZMzBxZEdvNy9HemEzWC9iaEhmTzlIL1RjNGdUWGJFQ0FpbjNV?= =?utf-8?B?a1I5dnBEbU56cUtWYk1Jc2lXWHNoQWw5WWs4OVNPQktqandBTWc5M0FKbENI?= =?utf-8?B?djhLVzJ3ZVpvclM4SHJxelNZMFc2SDlMSHhMWEhZUENsNWdPVE54S1U0OXJu?= =?utf-8?B?aWUxOElyTmhLeUc0dEpKQT09?= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DB4PR09MB5847.eurprd09.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230040)(376014)(366016)(1800799024)(13003099007)(8096899003)(7053199007); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?d3RWdjh0dzNDL0RWZDJhYVBEVmxMSG9WdjJUMnFaTCthc1YydXJjaFhIRGVv?= =?utf-8?B?UnhrZ3E2cUw0M1FsRGdMOW90eVZZNjZyTGo3SWFZSW53TWJuL0puM0VIajJK?= =?utf-8?B?QXF0WnJTQzRWZVgrbEtMWmFYNHZmYkEzc01UMnFxNmMxRGFCUmZHTkQrYjlh?= =?utf-8?B?VUZCd0htVG1ZTm9qZmZqaFFNZ2lCTlJGVVNSa3NpSzBYeEdoMkROTVJFQmk4?= =?utf-8?B?YUkvQmtWTHh5eG5JQjdVSW1lWUs1NVVxbjdCWDBsZUtiTDZ3c1hDcHY3OFJS?= =?utf-8?B?M0RuV3gvTkFiMlJFbElKR21MUWdnTnd4VDdsVE5YSEYyaklISlJVZVIwd1pO?= =?utf-8?B?OUhZdEtpZW5jY2pXeUd1TnJYTkUwK2lIVysvZlUxQVhVa2NiUlorU0l1aStj?= =?utf-8?B?S1FYTTE0V2JmQ1N4YWpLM213V0oyekN0cWhuRVhrK2wvVktyRFBwOUl6dTgy?= =?utf-8?B?dUdLSUt4NHc3MkpJUSs0cU5qUFd6cTB1UmE1RzduM2t2WjcxYTR2cVBEYjAv?= =?utf-8?B?MEpqKzFZZk1rcHZXOVkyTnZRRXVVdTJPS0FXU1JNNm9jRm5GRmY3WGJVVitG?= =?utf-8?B?aWhxVDE2UExSc2VZOTZaQmZ6ZmFjdlg4U1pnNklWeWd2TStSbHRra0pJS3pI?= =?utf-8?B?MnNkbXJlNmwxeHVDamg3Rk9nWmhCdkIrK0NRVFVDaTdTT2VoRVJ5dFFxUU5Z?= =?utf-8?B?Y0NTTURyc092T2FHVXFFTWZDUXVNQ0FoU1ArR29BNlByL1RSUTBUU0Z4Q29F?= =?utf-8?B?S3pSd3ZOVy9ZVlBkS0pZRWdRSml1UlREeXRNVUpTTjBSYTh3ZkNzWFNHbmZO?= =?utf-8?B?cS9ZSmRWYTVGWXlFWjdDTEtpeHJlUG05TlorRUx6UGhhYkUzVGdLUTRmTHRv?= =?utf-8?B?SUZ0NXVLZUlQTXRoeXpYdkhlS0dNai9nQjFoM2hCckhZR3U5YldsZ3o4b3c5?= =?utf-8?B?cEZZYzMrNjdTclQ2WUxZNmV5aTJGRm9DUEh3dHMxN2pKK0haN3Z6a2lLQ1Ri?= =?utf-8?B?RHFwVC9rSFY1Yk40ZUpiYlFaaHNuMDB1dmowbEhkZnBKWDRiZGViMWgvd2RF?= =?utf-8?B?d3N1bmFHc2c1Tkg5ZUZ1RUErcWJqdEMxRXZOVUdLb0M1M3YvYm8vTzcyMmRW?= =?utf-8?B?d3IxSWhucU5aWkE1S01SR0R3QjhVUEsrZFkrbm9Db1AzTDdObHZEQ2VUTmN0?= =?utf-8?B?d1k4ckV6aEo1blJHN1oxamE5QmZYVjVNa1YySEtGUEFueWhMM3N4RkUxU2hL?= =?utf-8?B?T0hRL0pSY3cvSGpyR24rZ0VMVWxaZllDSDlYZ040eXp1emI3T3NjMXhzcEIv?= =?utf-8?B?eU5GR1dJdmFBOC9UYnEyUDVTT3RHZ2hqRWhWbjRENFZ0ZXZxYmNiMzE0REQ3?= =?utf-8?B?MDFTU1JaN2N5aS9ZNzU4ejJlelBTUmYzVDN2WVBibVZVcDR1aFBhTFptZ3kw?= =?utf-8?B?aEhUSFpRSlJLU1VDKzFuMjJ0SlloQWV4TVNvdGxjTlRTVTdIMXlsTXk0dktm?= =?utf-8?B?UFlnU2tWZStaakxLc0NmUm1IWU1nUWRyUXU0LzlDZTFvajJLMjJIYmVuUzNp?= =?utf-8?B?ZEExTllVekUyVVQ4NWFFUlprd29SZTc2Rk1KKzFkcWc4WUc4aDgyVEI4eG5L?= =?utf-8?B?Y1I1dzMzc2E0ZlAxeWdobGRMWXFKTTY4Q0E5V3ZNQ1dGU3NuMFZJWGROOCtC?= =?utf-8?B?SFVtT09POHBzcHZjMTZteDc3djNvZk5LdTgraTBWTVkvY0N4di9YRExxd3p0?= =?utf-8?B?a0d2SGVyVUo4NlJJOVlBUGhyNTN0ZmdpOUhxaTBFTWRYSGxRZ2lhd0diZ2NU?= =?utf-8?B?Wko1UURUZFBPVS90L3JwQUhwSDdjc1JXb0JRYnVPekFZZDlLL3lVaGlWbk1Z?= =?utf-8?B?ejYyT2lTZWFpZXJNZjhlM2l3MEhUcnhBb2dOenZISjVqOHNvZnJJV21SclVH?= =?utf-8?B?UDJuNUppK2EwU010ZGk5SVBaZkFjNExCRmVGazM0R1dnWWpxNzdzUkZpSHFK?= =?utf-8?B?V2xTN0JZbUZpQkNwcitNRmVXS2tUd2pqeU9idVNzbDFMZFBNTlRPMXY2SmZ6?= =?utf-8?B?ZkdSTXpIb0t1bWo5ZlFNa2RvZ0F4ODhLNk5BZDdsMWpTeGxKZThNeGpHQmtR?= =?utf-8?B?d0RrWWJOZE1kR3BvbFNFREVOcGxHMkxhQTlKV1N4WEo2MzMyVlBCZ2dSUjFY?= =?utf-8?B?ekE9PQ==?= X-OriginatorOrg: noa-archive.com X-MS-Exchange-CrossTenant-Network-Message-Id: 05625ca0-a03e-42a6-13a9-08dd477a661a X-MS-Exchange-CrossTenant-AuthSource: DB4PR09MB5847.eurprd09.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Feb 2025 13:21:58.2781 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: a676c8d7-7f39-4c99-a306-48abb76a98e9 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: +qHsZFxJgGf2juelMyG0PpuQPDRMYS3G+3e7s3mEBfN9UbbuLqyNRgSGPLanglTUsFgK0uKRO36bN6AqrDb8xw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: GV2PR09MB6325 X-Content-Filtered-By: Mailman/MimeDel 2.1.29 Subject: Re: [FFmpeg-devel] [PATCH v5 2/3] fftools/opt_common: add time and datetime log flags 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-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On 07/02/2025 12:27, Soft Works wrote: > >> -----Original Message----- >> From: ffmpeg-devel On Behalf Of >> Tobias Rapp >> Sent: Friday, February 7, 2025 11:42 AM >> To:ffmpeg-devel@ffmpeg.org >> Subject: Re: [FFmpeg-devel] [PATCH v5 2/3] fftools/opt_common: add >> time and datetime log flags >> >> On 07/02/2025 08:57, softworkz wrote: >> >>> From: softworkz >>> >>> This commit adds two logging flags: 'time' and 'datetime'. >>> >>> Usage: >>> >>> ffmpeg -loglevel +time >>> >>> or >>> >>> ffmpeg -loglevel +datetime >>> >>> Setting av_log_set_flags(0) in term_exit in ffmpeg.c prevents >>> timing to be printed when exiting. >>> >>> Signed-off-by: softworkz >>> --- >>> fftools/ffmpeg.c | 1 + >>> fftools/opt_common.c | 12 ++++++++++++ >>> 2 files changed, 13 insertions(+) >>> >>> diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c >>> index dc321fb4a2..f4c717afaa 100644 >>> --- a/fftools/ffmpeg.c >>> +++ b/fftools/ffmpeg.c >>> @@ -130,6 +130,7 @@ static void term_exit_sigsafe(void) >>> >>> void term_exit(void) >>> { >>> + av_log_set_flags(0); >>> av_log(NULL, AV_LOG_QUIET, "%s", ""); >>> term_exit_sigsafe(); >>> } >> If I understand the purpose of AV_LOG_QUIET correctly the correct way >> would be to skip writing time/datetime information in the log writer >> itself (part of patch #1 in this patch-set) if level is <= >> AV_LOG_QUIET, >> rather than clearing the flags here. >> >> Regards, >> Tobias > > Hi Tobias, > > my understanding of AV_LOG_QUIET would be that when program code logs a message with that level, it will be printed even when the user has "-loglevel quiet". > I would wonder why such log messages shouldn't be prefixed with the time (normally). > To me it appears that the special case is rather the log invocation on exit which prints - well: nothing. > > I'd rather question why it is not printing the log level for quiet. Is it really intended or was it only done to make it possible to print this empty message on exit? > > Do I see it wrong? Then I'll change it of course. > > Thanks a lot for reviewing! > > sw The documentation for AV_LOG_QUIET in log.h says "Print no output" and from grepping through code it seems the implementation matches that definition. The purpose of av_log(NULL, AV_LOG_QUIET, "%s", "") is described with the AV_LOG_SKIP_REPEATED flag, see https://ffmpeg.org/doxygen/trunk/group__lavu__log.html#ga6cdf5cd331b17e80e8308124f05a6db8 . So I suggest to just add a "(level > AV_LOG_QUIET)" clause to the added lines in forma_line(), similar to how it is done with the loglevel prefix in that function some lines below: if (*print_prefix && (level > AV_LOG_QUIET) && (flags & (AV_LOG_PRINT_TIME | AV_LOG_PRINT_DATETIME))) format_date_now(&part[4], flags & AV_LOG_PRINT_DATETIME); Regards, Tobias _______________________________________________ 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".