MIME-Version: 1.0
References: 
 <CALDaNm16knMFtcqyAG3XYSkyagmVXfhaR0T=hau8UTAU0+eLQQ@mail.gmail.com>
 <CAHGQGwEuEd2StEYuP9vj+jCFT=hQbX8xcQeq8=Kj8r9xT3vmyg@mail.gmail.com>
 <CALDaNm0zJuUon52OEx5SpMoP8LEGJh+=PyYqwmcD7Zh97_dnJg@mail.gmail.com>
 <CAHGQGwGFP1AEQ1+=95LrVpUUOOPeebi13aEXXebSZ7bvWVT=wA@mail.gmail.com>
 <CALDaNm0MO0qSwN2tbEyzogocdV4COwGQs9_OD7r+P4GaWnac=g@mail.gmail.com>
 <CAAJ_b975xBx_No+i69F5EAcRgDaT-G0h=XPFrqL1n_zBSwKSRQ@mail.gmail.com>
In-Reply-To: 
 <CAAJ_b975xBx_No+i69F5EAcRgDaT-G0h=XPFrqL1n_zBSwKSRQ@mail.gmail.com>
From: Fujii Masao <masao.fujii@gmail.com>
Date: Thu, 16 Apr 2026 23:06:25 +0900
Message-ID: 
 <CAHGQGwEbDkqcDU=zHeQFC-dyBDv7UM108bFCqd7gkx3b=UROig@mail.gmail.com>
Subject: Re: Use XLogRecPtrIsValid() instead of negated XLogRecPtrIsInvalid
To: Amul Sul <sulamul@gmail.com>
Cc: vignesh C <vignesh21@gmail.com>,
	PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: 
 <https://www.postgresql.org/message-id/CAHGQGwEbDkqcDU%3DzHeQFC-dyBDv7UM108bFCqd7gkx3b%3DUROig%40mail.gmail.com>
Precedence: bulk

On Thu, Apr 16, 2026 at 9:31=E2=80=AFPM Amul Sul <sulamul@gmail.com> wrote:
> The outer parentheses do not seem to be needed, as
> XLogRecPtrIsInvalid() already includes them.
>
> Other than that, the v3 patch looks good to me.

I've updated the patch based on Amul's suggestion and pushed it. Thanks!

Regards,

--=20
Fujii Masao