#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 eschnett):
Ah, that's where these errors were coming from... I think I "remedied" them by allowing duplicate symbols in the linker.
In this case, I think that adding -fgnu89-inline to the Cactus options is the best short-term solution. In the long term, we may want to switch to the new inline semantics; this would include ensuring that shared C/C++ header files behave correctly, and updating the C Cactus thorns which use inline.
#define inline was always part of Cactus; this is standard procedure in GNU autoconf, and was not introduced to correct a C99 problem.