Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#12537 closed Bugs (fixed)

Undefined BOOST_WORKAROUND in boost/math/tools/tuple.hpp

Reported by: andreas.schuh.84@… Owned by: John Maddock
Milestone: To Be Determined Component: math
Version: Boost 1.62.0 Severity: Problem
Keywords: Cc:

Description

This workaround has been added in 1.62. The issue was not present with Boost 1.61.

/usr/local/include/boost/math/tools/tuple.hpp:10:60: error: token is not a valid binary operator in a preprocessor subexpression #if !defined(BOOST_NO_CXX11_HDR_TUPLE) && !BOOST_WORKAROUND(BOOST_GCC_VERSION, < 40500)

The particular line is here: https://github.com/boostorg/math/blob/70251d14429c11578a9940cf7b9fb7fe13865421/include/boost/math/tools/tuple.hpp#L10

Change History (3)

comment:1 by anonymous, 6 years ago

I am wondering if users of Boost math library are required to include "boost/math/config.hpp" before? This file seems to include "boost/detail/workaround.hpp"...

https://github.com/boostorg/math/blob/develop/include/boost/math/tools/config.hpp

comment:2 by John Maddock, 6 years ago

Resolution: fixed
Status: newclosed

It's a bug, fixed in https://github.com/boostorg/math/commit/3370ea2f794b8e36214eb4fadfccee01e109eb81

Note however that this is an undocumented internal header, if at some time we stop supporting pre-c++11 compilers then this header will probably disappear.

comment:3 by andreas.schuh.84@…, 6 years ago

Thanks for the quick reply. Fair point, I've updated our code to use std::tuple instead.

Note: See TracTickets for help on using tickets.