Peter Geoghegan <pg@bowt.ie> writes:
> It also inlines (in the second patch) by marking the _bt_compare
> definition as inline, while not changing anything in nbtree.h. I
> believe that this is portable C99 -- let's see what CF Tester thinks
> of it.
Boy, I'd be pretty darn hesitant to go there, even with our new
expectation of C99 compilers. What we found out when we last experimented
with non-static inlines was that the semantics were not very portable nor
desirable. I've forgotten the details unfortunately. But why do we need
this, and what exactly are you hoping the compiler will do with it?
regards, tom lane