'ifinite' was not declared in this scope

Question

I'm trying to compile shogun toolbox and I'm getting this fault

    C:/shogun-3.0.0/shogun-3.0.0/src/shogun/../shogun/mathematics/Math.h: In static
    member function 'static int shogun::CMath::is_finite(double)':
    C:/shogun-3.0.0/shogun-3.0.0/src/shogun/../shogun/mathematics/Math.h:1255:20: er
    ror: 'ifinite' was not declared in this scope
    return ifinite(f);

function itself looks like this.

        inline static int is_finite(double)
        {      
        #if defined(isfinite) && !defined(SUNOS)
        return ifinite(f);
        #else
        return finite(f);
        #endif
        }

I believe similar is described here: http://www.alecjacobson.com/weblog/?p=1768, but I'm not sure as I don't include cmath. Any idea what it can be?

Shouldn't the function's name be `isfinite` and not `ifinite`? — Morwenn, Nov 21 '13 at 13:29
I think it is unlikely that OP typed both the source and the error messages and made the some typo in three places. This would have to be an error in the source. But it is not in the 3.0.0 source that I just downloaded. So how did it get there? — Eric Postpischil, Nov 21 '13 at 15:21

score 2 · Answer 1 · answered Nov 21 '13 at 13:32

2

Function is isfinite, not ifinite.

You don't include <cmath> but according to Shogun source here, it does include both <cmath> and <math.h> in the wrong order:

#include <shogun/base/SGObject.h>
#include <shogun/lib/common.h>
#include <cmath>                             <<<<<<
#include <shogun/mathematics/Math.h>
#include <shogun/mathematics/lapack.h>
#include <shogun/io/SGIO.h>

#include <stdio.h>
#include <stdlib.h>
#include <math.h>                            <<<<<<

So you are supposed to use std::isfinite.

answered Nov 21 '13 at 13:32

Jack

131,802
30
241
343

changing the order in Math.cpp didnt help. – Krzysztof Fajst Nov 21 '13 at 14:07
changing all calls in math.h like – Krzysztof Fajst Nov 21 '13 at 14:07
//if (!CMath::is_finite(p)) if (!std::is_finite(p) return q; //if (!CMath::is_finite(q)) if (!std::is_finite – Krzysztof Fajst Nov 21 '13 at 14:11
dont help also. Sorry, dont know how to add the code in the comment, – Krzysztof Fajst Nov 21 '13 at 14:12
There is no "wrong order" in which `` and `` can be included. Usually, `` includes ``, so there can never be a conflict anyways... – rubenvb Nov 21 '13 at 15:23

score 2 · Answer 2 · answered Nov 21 '13 at 15:20

I just downloaded shogun-3.0.0 from here, and there is no occurrence of the string “ifinite” anywhere in the source. The definition of is_finite in Math.h is:

        /// checks whether a float is finite
        inline static int is_finite(double f)
        {
#if defined(isfinite) && !defined(SUNOS)
            return isfinite(f);
#else
            return finite(f);
#endif
        }

If the errors and source text you entered into the question are correct, perhaps the sources you have were corrupted. You should download the source and try again.

'ifinite' was not declared in this scope

2 Answers2