lmi
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lmi] [PATCH] Simple fix for yet another clang warning in Boost code


From: Vadim Zeitlin
Subject: [lmi] [PATCH] Simple fix for yet another clang warning in Boost code
Date: Sun, 31 Jul 2016 23:41:46 +0200

 Hello,

 I think I haven't built lmi with clang since rate table stuff was added in
4f738399b2a45029ba1535cad186e0e4d5cda67e, but now that I try to do it I'm
getting

        In file included from .../rate_table.cpp:33:
        In file included from .../3rdparty/boost_1_33_1/boost/optional.hpp:15:
        .../3rdparty/boost_1_33_1/boost/optional/optional.hpp:668:34: error: 
unused parameter 'x' [-Werror,-Wunused-parameter]
        bool operator == ( none_t const& x, optional<T> const& y )
                                         ^
        .../3rdparty/boost_1_33_1/boost/optional/optional.hpp:673:33: error: 
unused parameter 'x' [-Werror,-Wunused-parameter]
        bool operator < ( none_t const& x, optional<T> const& y )
                                        ^
        2 errors generated.
        Makefile:4569: recipe for target 'rate_table.o' failed
        make[1]: *** [rate_table.o] Error 1

and the build fails. I don't understand why doesn't g++ give the same
warning because the parameter is clearly unused, but it seems logical that
clang does, so I'd like to fix this by locally disabling this warning for
it as done by https://github.com/vadz/lmi/pull/45

 The only alternative would be to patch Boost headers to remove the
"none_t" parameter name, please let me know if you think this would be
preferable.

 Thanks in advance,
VZ


reply via email to

[Prev in Thread] Current Thread [Next in Thread]