Horton
Menu

Re: Remove unnecessary casts in printf format arguments

Peter Eisentraut <peter@eisentraut.org>

View thread
From: Peter Eisentraut <peter@eisentraut.org>
To: Bertrand Drouvot <bertranddrouvot.pg@gmail.com>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2025-12-09T06:45:41Z
Lists: pgsql-hackers 
On 04.12.25 14:51, Bertrand Drouvot wrote:
> On Thu, Dec 04, 2025 at 01:19:07PM +0100, Peter Eisentraut wrote:
>> Attached are some patches to remove unnecessary casts in printf format
>> arguments.  Many of these are apparently left over from before use of
>> %zu/%zd and PRIu64/PRId64 were possible.  Additionally, there are a few
>> where the cast just cast to the type the input already had, or it just
>> uselessly cast to a different type instead of using the right format
>> argument.
> 
> I did look at all of them and did not see any issues, so that LGTM.

committed, thanks

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

  1. Clean up int64-related format strings

  2. Remove unnecessary casts in printf format arguments (%zu/%zd)

  3. Remove useless casts in format arguments