public inbox for [email protected]  

help / color / mirror / Atom feed

From: John Naylor <[email protected]>
To: Jianghua Yang <[email protected]>
Cc: [email protected]
Subject: Re: BUG: test_ginpostinglist second itemptr check is a no-op due to copy-paste error
Date: Tue, 24 Mar 2026 17:30:14 +0700
Message-ID: <CANWCAZZbjetZPb8XhCvNRNo8Ji_qaowq1VT4hhyoLiBAGaJo3w@mail.gmail.com> (raw)
In-Reply-To: <CANWCAZbHJmmais=W-4BmP3ZTdGSUSxKMV07idje3CfKtJirxxw@mail.gmail.com>
References: <CAAZLFmSOD8R7tZjRLZsmpKtJLoqjgawAaM-Pne1j8B_Q2aQK8w@mail.gmail.com>
	<CANWCAZbHJmmais=W-4BmP3ZTdGSUSxKMV07idje3CfKtJirxxw@mail.gmail.com>

On Tue, Mar 24, 2026 at 8:59 AM John Naylor <[email protected]> wrote:
>
> On Tue, Mar 24, 2026 at 8:45 AM Jianghua Yang <[email protected]> wrote:
> >   The attached patch fixes the second check to compare orig_itemptrs[1]
> >   with decoded_itemptrs[1] and reports (blk, off) as the expected
> >   value in the error message.
>
> Seems right to me, thanks for the patch. I'll take care of this.

This is done. It seems unlikely that this test would need to be
changed in back branches, but I went ahead and backpatched it anyway
just in case.

--
John Naylor
Amazon Web Services

view thread (3+ messages)

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: BUG: test_ginpostinglist second itemptr check is a no-op due to copy-paste error
  In-Reply-To: <CANWCAZZbjetZPb8XhCvNRNo8Ji_qaowq1VT4hhyoLiBAGaJo3w@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox