Обсуждение: Remove INT64_HEX_FORMAT and UINT64_HEX_FORMAT

Поиск
Список
Период
Сортировка

Remove INT64_HEX_FORMAT and UINT64_HEX_FORMAT

От
Peter Eisentraut
Дата:
These were introduced (commit efdc7d74753) at the same time as we were 
moving to using the standard inttypes.h format macros (commit 
a0ed19e0a9e).  It doesn't seem useful to keep a new already-deprecated 
interface like this with only a few users, so remove the new symbols 
again and have the callers use PRIx64.

(Also, INT64_HEX_FORMAT was kind of a misnomer, since hex formats all 
use unsigned types.)

[should be backpatched to PG18]
Вложения

Re: Remove INT64_HEX_FORMAT and UINT64_HEX_FORMAT

От
Nathan Bossart
Дата:
On Sat, Aug 02, 2025 at 11:09:16AM +0200, Peter Eisentraut wrote:
> These were introduced (commit efdc7d74753) at the same time as we were
> moving to using the standard inttypes.h format macros (commit a0ed19e0a9e).
> It doesn't seem useful to keep a new already-deprecated interface like this
> with only a few users, so remove the new symbols again and have the callers
> use PRIx64.

That seems fine to me.

-- 
nathan



Re: Remove INT64_HEX_FORMAT and UINT64_HEX_FORMAT

От
Thomas Munro
Дата:
On Sun, Aug 3, 2025 at 6:25 AM Nathan Bossart <nathandbossart@gmail.com> wrote:
> On Sat, Aug 02, 2025 at 11:09:16AM +0200, Peter Eisentraut wrote:
> > These were introduced (commit efdc7d74753) at the same time as we were
> > moving to using the standard inttypes.h format macros (commit a0ed19e0a9e).
> > It doesn't seem useful to keep a new already-deprecated interface like this
> > with only a few users, so remove the new symbols again and have the callers
> > use PRIx64.
>
> That seems fine to me.

+1



Re: Remove INT64_HEX_FORMAT and UINT64_HEX_FORMAT

От
Peter Eisentraut
Дата:
On 04.08.25 05:03, Thomas Munro wrote:
> On Sun, Aug 3, 2025 at 6:25 AM Nathan Bossart <nathandbossart@gmail.com> wrote:
>> On Sat, Aug 02, 2025 at 11:09:16AM +0200, Peter Eisentraut wrote:
>>> These were introduced (commit efdc7d74753) at the same time as we were
>>> moving to using the standard inttypes.h format macros (commit a0ed19e0a9e).
>>> It doesn't seem useful to keep a new already-deprecated interface like this
>>> with only a few users, so remove the new symbols again and have the callers
>>> use PRIx64.
>>
>> That seems fine to me.
> 
> +1

done