Opened 13 years ago

Closed 13 years ago

#431 closed discussion (fixed)


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


what is that macro doing?

It is checking if that header is available, and defines a PP macro in config.h. BUT we are not using config.h in any file, and shouldn't we at least define a ACTION-IF-NOT-FOUND. Or why are we doing this check?

Change History (2)

comment:1 Changed 13 years ago by Peter

Milestone: yat 0.x+yat 0.5

This actually includes all these macros (except AC_PROG_LIBTOOL)

# Checks for header files.

# Checks for typedefs, structures, and compiler characteristics.

# Checks for library functions.
AC_CHECK_FUNCS([pow sqrt])

Can I just remove them, or should we make configure complain if something is missing? I think it is stupid to have macros that just add info tho config.h which is never used. According to svn praise the macros were added in r71 and as far as I can see there was no reason but inexperience.

comment:2 Changed 13 years ago by Peter

Resolution: fixed
Status: newclosed

(In [1459]) Cleaning up in 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.