On Mon, Sep 25, 2023 at 1:18 PM Alexander Korotkov <aekorot...@gmail.com>
wrote:

> On Mon, Sep 25, 2023 at 12:58 PM Pavel Borisov <pashkin.e...@gmail.com>
> wrote:
> > One more doubt about naming. Calling function
> > _bt_checkkeys(IndexScanDesc scan, IndexTuple tuple, int tupnatts,
> > ScanDirection dir, bool *continuescan, bool requiredMatchedByPrecheck)
> > as
> > (void) _bt_checkkeys(scan, itup, indnatts, dir,
> > &requiredMatchedByPrecheck, false);
> > looks little bit misleading because of coincidence of names of 5 and 6
> > arguments.
>
> I've added the comment clarifying this argument usage.
>

Fixed typo inficating => indicating as pointed by Pavel.
Peter, what do you think about the current shape of the patch?

------
Regards,
Alexander Korotkov

Attachment: 0001-Skip-checking-of-scan-keys-required-for-direction-v7.patch
Description: Binary data

Reply via email to