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 A639A4DF9B
	for <ffmpegdev@gitmailbox.com>; Thu,  6 Mar 2025 10:08:42 +0000 (UTC)
Received: from [127.0.1.1] (localhost [127.0.0.1])
	by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 82F4D68F2C9;
	Thu,  6 Mar 2025 12:08:38 +0200 (EET)
Received: from nef.ens.fr (nef2.ens.fr [129.199.96.40])
 by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id B987868E821
 for <ffmpeg-devel@ffmpeg.org>; Thu,  6 Mar 2025 12:08:32 +0200 (EET)
X-ENS-nef-client: 129.199.129.80 ( name = phare.normalesup.org )
Received: from phare.normalesup.org (phare.normalesup.org [129.199.129.80])
 by nef.ens.fr (8.14.4/1.01.28121999) with ESMTP id 526A8Vhs000943
 for <ffmpeg-devel@ffmpeg.org>; Thu, 6 Mar 2025 11:08:32 +0100
Received: by phare.normalesup.org (Postfix, from userid 1001)
 id B73822EFE5; Thu,  6 Mar 2025 11:08:31 +0100 (CET)
Date: Thu, 6 Mar 2025 11:08:31 +0100
From: Nicolas George <george@nsup.org>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Message-ID: <Z8l0H8xQNhwcB5sR@phare.normalesup.org>
References: <pull.59.ffstaging.FFmpeg.1741189106311.ffmpegagent@gmail.com>
 <Z8hwUZUibUe5Ff4g@phare.normalesup.org>
 <DM8P223MB0365975894E9FE6F7424880DBACB2@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM>
MIME-Version: 1.0
Content-Disposition: inline
In-Reply-To: <DM8P223MB0365975894E9FE6F7424880DBACB2@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM>
X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.4.3
 (nef.ens.fr [129.199.96.32]); Thu, 06 Mar 2025 11:08:32 +0100 (CET)
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 <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>
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/Z8l0H8xQNhwcB5sR@phare.normalesup.org/>
List-Archive: <https://master.gitmailbox.com/ffmpegdev/>
List-Post: <mailto:ffmpegdev@gitmailbox.com>

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.

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