#484: CCTK_CHECK_C_INLINE can define away the inline keyword ---------------------+------------------------------------------------------ Reporter: rhaas | Owner: Type: defect | Status: new Priority: minor | Milestone: Component: Cactus | Version: Resolution: | Keywords: ---------------------+------------------------------------------------------
Comment (by rhaas):
Including cctk.h last might not not always be possible. Eg. if one wants to include some header files (unistd.h [see the flesh] or cxxabi.h [the backtrace patches in Carpet/Intialize.cc]) only if they were detected by configure then cctk.h has to come before these header files (not an issue for string.h admittedly).
I have used the --multiple-definition flag (for other reasons) in the past (-gnu89-inline also fixes this issue). Does it check that the two definitions are actually identical? ie. that they are not two completely different functions whose names clash?
Anyhow I doubt it is worth spending much time on this until it is decided whether Cactus should adhere to C99 or the traditional behaviour. We know how to compile on each system.