On Thu, Aug 4, 2022 at 3:43 PM Andres Freund <and...@anarazel.de> wrote: > > We retain a HAVE_SHM_OPEN macro, because it's clearer to readers than > > something like !defined(WIN32). > > I don't like these. I don't find them clearer - if we really just assume this > to be the case on windows, it's easier to understand the checks if they talk > about windows rather than having to know whether this specific check just > applies to windows or potentially an unspecified separate set of systems. > > But I guess I should complain upthread...
Thanks for reviewing. For this point, I'm planning to commit with those "vestigial" macros that Tom asked for, and then we can argue about removing them separately later.
