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 D1C834DFF5 for ; Thu, 6 Mar 2025 20:56:59 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 37F8E68F0DE; Thu, 6 Mar 2025 22:56:56 +0200 (EET) Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12olkn2034.outbound.protection.outlook.com [40.92.21.34]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 3DC9268ECEF for ; Thu, 6 Mar 2025 22:56:49 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=scHnfY0YCjJeA8Jjes5ACk70japdwxEAqFTj6A7Oq1eKv+dZWDkJPsiJIw1l+kpNpSlx5XfKV3IdXEgkWbFPzyzcscvXoh0UysLGFFqiiGcsW8aDsBC1YKQVOv0nH82xDTZQa09ShEJc78QLc6WlenVJ6jNjdRFW0DGACMKkxR0lzjUEnOFgViM39+OKnbvk69B8IvdgmvuqDNkNPipuxR7RcVEWVHnXHpUEkUIfRLiIXjY/ySW2jgSgqGx60V+0EfHuG4bjUroysExlrAcmzJLtsbGUxpCzZVh5cafdaXfzGuwCg7QzmY/FAMR1BAoVYx59kT8m0dstUvam51aSBg== 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=cXwBJGLjrtxK42c+RJl2DtrKlqbVXQ1jWh+FcMME6eo=; b=YZn914o9L7/k8aMY+6zDjugjuGi7qjJ+E+2BvcSk9C7wRm39ObTkyZyCCg+J+KzlHuJ2uXLuLDpozcStzuG+4GTdE5H0iXJ+3i8kg8ponpEI84PNVQD0ItwqPgKXYuz4aTs8M9cRhhpkZMHWKwfgOtjpQG8kU61DSF8z0ajfdExq7ew/X+G798/Yu7mnOTB8sEYu7d12KAIn0iSJIraFNuOQYqsARLkODzuVbA8OLdatXIPsKQXnDOewSzdOXaL3MuljntCWzIjbr4jT1C4sF7XFz/MT0nnaKOSvvdcpON/N3rSj+L+xfL6FS4Rwpn7rihFT3QURik4SNrfROQkNwg== 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=cXwBJGLjrtxK42c+RJl2DtrKlqbVXQ1jWh+FcMME6eo=; b=nSEPCpbwn2NWXV60km58VZMmvi3NtqoyHbrDgwcoWmHmgoRuFGWxF+77zX3/XieIVKl3GAkpl4Xhx8aFZvJ/+gVJePLoz2E8RwVxZWfpDZgki0MaOCnOrH2KKooUNwAzMxxUyrqoWvFZA63Y4ERJnHoKTXBL6NSLSHpTkLh4S4F93CyEbkxxpPQkmIF5VXHl61bpLdWnRvBUjh2esQ5zmAfKuqvI5u549ouroEp3iG1fWJWLIFEg16V67Yr/2N+pEBc3HWkvG/mT24LYpEE4T/aU0XV8ONHxe8DV4bqU/3TPpbHjdVCcpweLlBiKDHc4yaZFjj4WXYYuYP6ZTnHD2w== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by LV1P223MB1340.NAMP223.PROD.OUTLOOK.COM (2603:10b6:408:2b1::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8511.16; Thu, 6 Mar 2025 20:56:46 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::bf09:8e9:b07f:98a7]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::bf09:8e9:b07f:98a7%7]) with mapi id 15.20.8511.017; Thu, 6 Mar 2025 20:56:46 +0000 From: Soft Works To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH] avutil/log: Replace addresses in log output with simple ids Thread-Index: AQHbjeSnJYtY2XIGVE+nxmGIv6NZhbNkre2AgAACIJCAATOWgIAAs2dA Date: Thu, 6 Mar 2025 20:56:46 +0000 Message-ID: References: In-Reply-To: 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_|LV1P223MB1340:EE_ x-ms-office365-filtering-correlation-id: 30fc4deb-0f6f-48c2-1c91-08dd5cf1686b x-microsoft-antispam: BCL:0; ARA:14566002|15080799006|19110799003|7092599003|8062599003|8060799006|461199028|3412199025|13041999003|440099028|102099032; x-microsoft-antispam-message-info: =?iso-8859-1?Q?Zp7k9wUgUH6tzS5Kf+rwFAtPgW3Ws779wvDIYhIkk4VDeoLORo3Ww19Igj?= =?iso-8859-1?Q?ao+X88ED4bPxslSNFYkXPCut3LOl8Du0BROX5/IGxe3iFpHSFh3v8s7qOg?= =?iso-8859-1?Q?JvDh+ahlbiYrELSaCDgioHCK3cYMwYVTKZfMmUTscUHvL+zkk7WVT1Frzf?= =?iso-8859-1?Q?/g79uhIZU8leoIw16bNm1ClH8Q8y0b7iOtp8QV8V8l5FxVqlXOlif18i1Y?= =?iso-8859-1?Q?vbtb2PnDyAvY0OBYwqu9h5w5nDu2ySXF8zpX8pji/3pIH3LVLwmQ0g0NUW?= =?iso-8859-1?Q?SFEhGkpufMkrCOQNFXXaXQp9CowtQCaRPMDPkaMP1PdtK8NqgCb0+qbOGu?= =?iso-8859-1?Q?ezAedYgxFLtRqwcXRq8PmBbv4ZXFcSHdU/uGvyQq8uXI35WkTRiuRFn6JC?= =?iso-8859-1?Q?7sR5HOjsloINHLUQHePk7f5PQcRvbHcre1XMiBKg9K2KKMl8sfA7nR84u9?= =?iso-8859-1?Q?9UAzP5UIgUxP8FlkFfvPesghK7fGdCuC0t1ZhVnGQM0nruSbWJC/exEVqX?= =?iso-8859-1?Q?/nG/6oYN7EOTUQCaox861i8w8BIiEr9yWuFwHSgdBTd233YVJXMZaYtFM1?= =?iso-8859-1?Q?SDAZ0f1l/zdP2mQEDRu5xFG7PEAeUKuxDOqmia+t8r1OLnwG+jY9M2lOYb?= =?iso-8859-1?Q?3E2LQg8dnWLMVvfXG4oJxiTTM97fswiWqeLrP/LOkAJtXWMGo3UaSNaf4f?= =?iso-8859-1?Q?anU+1Ybx+U5q5YpEe0iOdQ7a5JCLQyTBkgHuY+4QcFSdHAZgDwpv/TFZtZ?= =?iso-8859-1?Q?IYKBSgdCyOL69S8mMaN1wm9P4xd3jxJdVgWHBllF0MRcu9lWb41Bxl5e0L?= =?iso-8859-1?Q?0AgvpwJ/bfKZBWDkk2s5ss6fUsIeGB+xV5GUMN2UNCW1topQmRhNm79T1h?= =?iso-8859-1?Q?0KkFxhrqq0sz8M5NSarlX5g50B1fuEym2Q8OcYfFOKDGrVrcXWOYgnvK4U?= =?iso-8859-1?Q?WpDZab2Kix494i/BLAWUD9f/RwB4m/FDIhAWHv0i3ru0DoHhf3x9PMijRo?= =?iso-8859-1?Q?veCB45BNzv6UNJO/536iNgTeVXIpsEDGipGDY8ueGEtDCLakPHsTiqc7TA?= =?iso-8859-1?Q?ntVMrPkLQfxMELKP11aqjIGe+aaIfMUnnS+R0yUKUhUzj62wGWEd0Eg7VY?= =?iso-8859-1?Q?uN+MVh/Z5aDjYYAgoDinVrWQroInY=3D?= x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?dRD4/tSoO3sRQy36t9XOYQ3kF/RSc6j5sk0WnhZ+pA0GF5Vp7vomI6U8YX?= =?iso-8859-1?Q?qKv+jHeDxW0ZuYspFpUiam2/6QBl5jgKlui3Qd6qb6zEQvdQOLGEsIDKhT?= =?iso-8859-1?Q?UydFthmYMub69+hmzrpbzvSYTiEVdLOgdKexZRDdOKLb9555rTaHCSDRzt?= =?iso-8859-1?Q?zuVijCY7MZ4SiBc4em5TdShap0SL1ORPc63HbEEhx1OmdtIt+0wcpGA0R7?= =?iso-8859-1?Q?8HsZg+G4whlPOau1+XW5RwuoZ1WNeicykVoYaAlKc7V5U46jXU00g2TIBI?= =?iso-8859-1?Q?hWVOd9ClJTEQtpoCCIrERYxYZk+mfQwfRlLpPJWVSRLmiI39JojwRxS8nb?= =?iso-8859-1?Q?+P15n58PSQivaTzomxOQK5Mye1uAa6KtAJFecHaqV1l5STwb2F7NLxAV0D?= =?iso-8859-1?Q?8NtyqemaKh6IzFwXpHLw5oVX2O9liQA/mj0X2uj2Bmi7A+43jO+NE+ZlwF?= =?iso-8859-1?Q?Q9QYnxKmkWQVULpti465YhDfPzXW0jIEqcsyYBPPlZFPSHqyF5emOto7i2?= =?iso-8859-1?Q?jW6yMQSGBYNEoz3i9NYyyxpAjtHm9LAPN+YjqgABOEhcb+9Ym7q/ucgCUW?= =?iso-8859-1?Q?6cNIXmX7IRQrAWJK35pbkaiWMSPUBHYEFOPiOvNU7vdOofklOzttkHDpJZ?= =?iso-8859-1?Q?HOoIF83yamRYY5lD6kql3TuuY8Qbvm3F3Qi3+ZPom4I++fIYOxXrjdIaa8?= =?iso-8859-1?Q?M72RV6AQKOgN2Noagy8TfWWtcZbymsJV1sbt/7+fgSUaRszqxsrn+mcUMf?= =?iso-8859-1?Q?c36XHUks3W/F9rNdAdvXBZAP8fE4DQVf960tPfS0P5Lv0sdJHxwBmNq0n6?= =?iso-8859-1?Q?BLsr06GsTVGdvPc2XHsfiekkdD1n/omf8m7GOM5f7voPdtC6sqDyBs9lCT?= =?iso-8859-1?Q?hbFp5HDQCPadalO+6vHF9SBP1MyLTJRAWqMkDj1PTrgJ0tyCvFb+tQ6HVI?= =?iso-8859-1?Q?ryNjnqQiNgwjdf+jxD7ggFqv1kTdhxqpOBl8qC4P2iqn4mWMX5gqY74G4A?= =?iso-8859-1?Q?uOjG5ZK0CUwQe4KK5Z87YipYFTkQfXgyAD2R67KzKS9LxTz5g0eRWeONak?= =?iso-8859-1?Q?CW1KpNctoVlOM+ho6H0T36KHAjlRcMOPft54JjL6h2gigvqI6y6HK8Gw1E?= =?iso-8859-1?Q?IYEM9VXjlLeXKVJ/UuDhVV8hugoSaTuKBvUm/uQqq6LPWWYZXIEAvdm5/K?= =?iso-8859-1?Q?hV3hVvUroqQm/xvsb4Zy3/j9rQrq/05K7XtPKedQIqpRMVD+y6OZkLQ97Z?= =?iso-8859-1?Q?TuMNXGhx7/F3REd69Chg6HSqMI4MAQ7KNA/23X42w=3D?= MIME-Version: 1.0 X-OriginatorOrg: sct-15-20-4755-11-msonline-outlook-1ff67.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: 30fc4deb-0f6f-48c2-1c91-08dd5cf1686b X-MS-Exchange-CrossTenant-originalarrivaltime: 06 Mar 2025 20:56:46.4586 (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: LV1P223MB1340 Subject: Re: [FFmpeg-devel] [PATCH] avutil/log: Replace addresses in log output with simple ids 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-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable 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 George > Sent: Donnerstag, 6. M=E4rz 2025 11:09 > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH] avutil/log: Replace addresses in log > output with simple ids > = > Soft Works (HE12025-03-05): > > Sorry. So - seriously: what would be your recipe then? > = > I see not just a little of non-trivial code for a very minor feature, > that might be a hint that it would be best to let it go. > = > Also, if somebody is debugging a program using the libraries, the > pointers are relevant for that program. For that reason, I think the > change is a bad idea in the library. > = > On the other hand, you could do that change in the fftools. The point > about pointers being relevant does not apply for them, and they can have > as much global state as they want. HI Nicolas, initially I failed to see the impact of the array being global, I tend to f= orget about direct usages of the libs, sorry about that. V3 of the patchset goes the route you are suggesting by introducing a callb= ack for formatting of the context prefixes, so the global state lives in ff= tools only. Thanks sw _______________________________________________ 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".