Re: Use XLogRecPtrIsValid() instead of negated XLogRecPtrIsInvalid - Mailing list pgsql-hackers

From Xiaopeng Wang
Subject Re: Use XLogRecPtrIsValid() instead of negated XLogRecPtrIsInvalid
Date
Msg-id 9cbbd8fc-4e46-495d-8260-a267196a1826@163.com
Whole thread
In response to Re: Use XLogRecPtrIsValid() instead of negated XLogRecPtrIsInvalid  (Fujii Masao <masao.fujii@gmail.com>)
List pgsql-hackers
在 2026/4/16 12:11, Fujii Masao 写道:
> On Mon, Apr 13, 2026 at 4:10 PM vignesh C <vignesh21@gmail.com> wrote:
>> I felt these also should be updated, the attached v2 version patch
>> includes the changes for the same.
> Thanks for updating the patch!
>
> -       applyPtr = (latestApplyPtr == InvalidXLogRecPtr) ?
> +       applyPtr = (XLogRecPtrIsInvalid(latestApplyPtr)) ?
>
> XLogRecPtrIsValid() should be used here, instead?
>
> Regards,
>

Yeah, I was about to raise the same comment, and Fujii-san beat me. I believe it should be XLogRecPtrIsValid().

Otherwise, the patch looks good to me.

Regards,
Xiaopeng Wang




pgsql-hackers by date:

Previous
From: Amit Langote
Date:
Subject: Re: [PATCH] Fix wrong comment in JsonTablePlanJoinNextRow()
Next
From: Masahiko Sawada
Date:
Subject: Re: First draft of PG 19 release notes