isnan() and isinf() checks

Alexander Neundorf neundorf at kde.org
Thu Apr 12 22:30:16 BST 2007


On Thursday 12 April 2007 20:58, Hasso Tepper wrote:
> Alexander Neundorf wrote:
> > Using "-lm" is no good idea, as Christian already stated.
>
> Mkay. Better now?
>
> > Please take a look at kdelibs/cmake/modules/MacroPushRequiredVars.cmake,
> > it offers two macros:
> >
> > # Usage:
> > # MACRO_PUSH_REQUIRED_VARS()
> > # SET(CMAKE_REQUIRED_DEFINITIONS
> >       ${CMAKE_REQUIRED_DEFINITIONS} -DSOME_MORE_DEF)
> > # CHECK_FUNCTION_EXISTS(...)
> > # MACRO_POP_REQUIRED_VARS()
>
> Thanks for info.
>
> > Can you please check whether check_symbol_exists() (from
> > Modules/CheckSymbolExists.cmake) does what you need instead of using
> > check_c_source_compiles() directly ?
>
> Indeed. I didn't realise that it does work for macros as well. Updated
> patch is attached.
>

Looks good, at least from the cmake side :-)

Alex
-- 
Work: alexander.neundorf AT jenoptik.com - http://www.jenoptik-los.de
Home: neundorf AT kde.org                - http://www.kde.org
      alex AT neundorf.net               - http://www.neundorf.net




More information about the kde-core-devel mailing list