Andrew Pinski <[EMAIL PROTECTED]> writes: | On May 24, 2005, at 12:01 AM, Zack Weinberg wrote: | > Use of bare 'inline' is just plain wrong in our source code; this has | > nothing to do with C++, no two C compilers implement bare 'inline' | > alike. Patches to add 'static' to such functions (AND MAKING NO OTHER | > CHANGES) are preapproved, post-slush. | That will not work for the cases where the bare 'inline' are used | because they are external also in this case. Now this is where C99 and | C++ differs at what a bare 'inline' means so I have no idea what to | do, except for removing the 'inline' in first place.
It suffices to make the function "static inline". See our current extensive usage. That works for both C and C++. -- Gaby
