Andres Freund <and...@anarazel.de> writes: > On 2017-03-14 19:34:12 -0400, Tom Lane wrote: >> It seems bizarre that you chose to spell the new configure symbol as >> HAVE__COMPUTED_GOTO rather than HAVE_COMPUTED_GOTO
> I went back-and-forth about this a number of times. We have a bunch of > symbols defined with HAVE__ as a prefix (and some with HAVE_GCC__) - and > more of the nearby code seems to use __ rather than _. I don't really > know why we started doing that, but it's far from new.. > Any idea why we introduce __ stuff? The nearby stuff is describing features that have a specific name that includes a leading underscore or two, like __builtin_unreachable(). That doesn't seem to apply here, so I wouldn't add extra underscores. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers