Opened 13 years ago

Closed 13 years ago

#430 closed enhancement (fixed)

AC_C_INLINE and -DHAVE_INLINE

Reported by: Peter Owned by: Peter
Priority: minor Milestone: yat 0.5
Component: build Version: trunk
Keywords: Cc:

Description

related to ticket:405

We use macro AC_C_INLINE to check if inline is available. Then we let CPPFLAGS include "-DHAVE_INLINE". I think we should use the information from the macro call to decide if can use "-DHAVE_INLINE". Alternatively, we can assume that inline is available everywhere, and thereby remove the call to AC_C_INLINE.

Actually, it may be that AC_C_INLINE is a C specific macro and then we should we remove it.

Change History (2)

comment:1 Changed 13 years ago by Peter

Milestone: yat 0.x+yat 0.5
Owner: changed from Jari Häkkinen to Peter
Status: newassigned

AC_C_INLINE is for C compilers and not useful for C++ (even avoided to be affectd by C++ compilers):

/* Define to `__inline__' or `__inline' if that's what the C compiler
   calls it, or to nothing if 'inline' is not supported under any name.  */
#ifndef __cplusplus
/* #undef inline */
#endif

So I will simply remove the macro.

comment:2 Changed 13 years ago by Peter

Resolution: fixed
Status: assignedclosed

(In [1459]) Cleaning up in configure.ac. Removed a couple of macros that were C specific or obsolete. Possibly we could keep tests for sqrt but then we need to use the test somehow (see discussion in ticket:430. Closes #430 and #431.

Note: See TracTickets for help on using tickets.