From: | Ranier Vilela <ranier(dot)vf(at)gmail(dot)com> |
---|---|
To: | Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Signed vs. Unsigned (some) |
Date: | 2022-02-13 20:19:46 |
Message-ID: | CAEudQApnxSWe1m_eLxSzYO+Ooe78SgeNe8P8d-jUFwqr_c=VGQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Em sex., 11 de jun. de 2021 às 23:05, Ranier Vilela <ranier(dot)vf(at)gmail(dot)com>
escreveu:
> Hi,
>
> Removing legitimate warnings can it be worth it?
>
> -1 CAST can be wrong, when there is an invalid value defined
> (InvalidBucket, InvalidBlockNumber).
> I think depending on the compiler -1 CAST may be different from
> InvalidBucket or InvalidBlockNumber.
>
> pg_rewind is one special case.
> All cases of XLogSegNo (uint64) initialization are zero, but in pg_rewind
> was used -1?
> I did not find it InvalidXLogSegNo!
> Not tested.
>
> Trivial patch attached.
>
After a long time, finally a small part is accepted and fixed.
https://github.com/postgres/postgres/commit/302612a6c74fb16f26d094ff47e9c59cf412740c
regards,
Ranier Vilela
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2022-02-13 20:24:03 | Re: Adding CI to our tree |
Previous Message | Tom Lane | 2022-02-13 20:16:58 | Re: Mark all GUC variable as PGDLLIMPORT |