[math] GCC 4.8+ warns unused local typedef...

[Chris Stylianou <chris5287@…>]

GCC 4.8+ warns unused local typedef...

airy.hpp:343
typedef ‘value_type’ locally defined but not used [-Wunused-local-typedefs]

airy.hpp:349
typedef ‘forwarding_policy’ locally defined but not used [-Wunused-local-typedefs]

airy.hpp:391
typedef ‘value_type’ locally defined but not used [-Wunused-local-typedefs]

airy.hpp:397
typedef ‘forwarding_policy’ locally defined but not used [-Wunused-local-typedefs]

beta.hpp:1334
typedef ‘evaluation_type’ locally defined but not used [-Wunused-local-typedefs]

beta.hpp:1352
typedef ‘evaluation_type’ locally defined but not used [-Wunused-local-typedefs]

Proposed fix attached.

[pmachata@…]

Another unused typedef is in math/distributions/inverse_gaussian.hpp

[pmachata@…]

There's also one suspicious (unused) typedef in libs/math/test/compile_test/test_compile_result.hpp:

template <class T1, class T2>
inline void check_result_imp(T1, T2)
{
   typedef int static_assertion[sizeof(T1) == 0xFFFF];
}

That's kind of reminiscent of static assert, but that should be something like [-(sizeof(T1) == 0xFFFF)] instead. In any case, it's not clear to me what the condition itself is meant to test.

[anonymous]

I'll get these fixed shortly, the one in test_compile_result.hpp is a static assert that should always fail. It also has to be implemented without including any other header (like static_assert.hpp) as those tests are designed to verify that all the needed headers have been included. I'll add a comment to that effect.

[Petr Machata <pmachata@…>]

In that case I'd go with something like sizeof(T1) != sizeof(T1), as that's guaranteed to fail. But I don't know the code, 0xFFFF may be just as impossible.

I guess the warning is no big deal, since this is for test suite only, but still, it would be nice to have the BOOST_STATIC_ASSERT_UNUSED_ATTRIBUTE copied over from static_assert.hpp.

[John Maddock]

(In [85131]) Fix gcc-4.8.x warnings. Fix some Mingw failures. Fixes #8855.

[John Maddock]

(In [85987]) Merge accumulated patches from Trunk. Refs #8384, Refs #8855, refs #9107, refs #9109, refs #8333, refs #8621, refs #8732, refs #8733, refs #8837, refs #8940, refs #9042, refs #9087, refs #9104, refs #9126.