Horton
Menu

Re: Remove unnecessary casts in printf format arguments

Bertrand Drouvot <bertranddrouvot.pg@gmail.com>

View thread
From: Bertrand Drouvot <bertranddrouvot.pg@gmail.com>
To: Peter Eisentraut <peter@eisentraut.org>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2025-12-04T13:51:31Z
Lists: pgsql-hackers 
Hi,

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.

Regards,

-- 
Bertrand Drouvot
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com

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