Boost C++ Libraries: Ticket #8855: [math] GCC 4.8+ warns unused local typedef...

attachment set

Thu, 18 Jul 2013 19:45:04 GMT

attachment → math.patch

attachment set

pmachata@… — Mon, 22 Jul 2013 14:18:34 GMT

attachment → boost-1.54.0-math-unused_typedef.patch

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

pmachata@… — Mon, 22 Jul 2013 14:26:46 GMT

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 — Mon, 22 Jul 2013 15:28:27 GMT

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.

Mon, 22 Jul 2013 23:02:03 GMT

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.

status changed; resolution set

John Maddock — Tue, 23 Jul 2013 11:09:50 GMT

status new → closed
resolution → fixed

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

John Maddock — Sat, 28 Sep 2013 16:19:56 GMT

(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.