On Fri, Jul 10, 2020 at 6:55 PM Andrey M. Borodin <x4...@yandex-team.ru> wrote: > Thanks! Fixed.
It's not a bug, but I think those 64 bit constants should be wrapped in UINT64CONST(), following our convention. I'm confused about these two patches: 0001 introduces gist_point_fastcmp(), but then 0002 changes it to gist_bbox_fastcmp(). Maybe you intended to keep both of them? Also 0002 seems to have fixups for 0001 squashed into it.
