On Mon, Nov 18, 2024 at 3:41 PM Heikki Linnakangas <hlinn...@iki.fi> wrote: > > On 19/11/2024 01:20, Masahiko Sawada wrote: > > I realized that building test_radixtree.c with TEST_SHARED_RT fails > > because it eventually sets RT_SHMEM when #include'ing radixtree.h but > > it's missing some header files to include. I've attached a patch to > > include necessary header files in radixtree.h to make it > > self-contained. > > +1. Please make sure the #includes are in alphabetical order.
Fixed. > > While we're at it, I noticed that lib/radixtree.h includes "postgres.h". > That's against our usual convention. Good catch. I've updated the patch accordingly. Regards, -- Masahiko Sawada Amazon Web Services: https://aws.amazon.com
v2-0001-Include-necessary-header-files-in-radixtree.h.patch
Description: Binary data
