On 16.05.24 16:45, Tom Lane wrote:
> Peter Eisentraut <peter(at)eisentraut(dot)org> writes:
>> In these cases, I think for
>> NotificationHash
>> ResourceOwnerData
>> WalSyncMethod
>> we can just get rid of the typedef.
>
> I have no objection to dealing with NotificationHash as you have here.
>
>> ReadBuffersFlags shouldn't be an enum at all, because its values are
>> used as flag bits.
>
> Yeah, that was bothering me too, but I went for the minimum delta.
> I did think that a couple of integer macros would be a better idea,
> so +1 for what you did here.
I committed this, and Michael took care of WaitEventExtension, so we
should be all clear here.