Horton
Menu

Thread

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

  1. Remove unnecessary casts in printf format arguments

    Peter Eisentraut <peter@eisentraut.org> — 2025-12-04T12:19:07Z

    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.

  2. Re: Remove unnecessary casts in printf format arguments

    Bertrand Drouvot <bertranddrouvot.pg@gmail.com> — 2025-12-04T13:51:31Z

    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

  3. Re: Remove unnecessary casts in printf format arguments

    Peter Eisentraut <peter@eisentraut.org> — 2025-12-09T06:45:41Z

    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