[ET Trac] [Einstein Toolkit] #484: CCTK_CHECK_C_INLINE can define away the inline keyword

[Einstein Toolkit]

#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.

-- 
Ticket URL: <https://trac.einsteintoolkit.org/ticket/484#comment:10>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit