Andres Freund <andres@2ndquadrant.com> writes:
> The reason I had the header declare DEFINE_ILIST_FUNCTIONS (or rather
> ILIST_USE_DEFINITION back then) instead of reusing USE_INLINE directly is that
> it makes it easier to locally change a "module" to not inlining which makes
> testing the !USE_INLINE case easier. Does anybody think this is worth
> something? I have no strong feelings but found it convenient.
Right offhand it doesn't seem like it really gains that much even for
that use-case. You'd end up editing the include file either way, just
slightly differently.
regards, tom lane