Opened 8 years ago

Closed 8 years ago

#10941 closed Bugs (fixed)

boost/multiprecision/cpp_int/misc.hpp uses boost::math::gcd (and lcm), but does not include common_factor_rt.hpp

Reported by: Peter Dimov Owned by: John Maddock
Milestone: To Be Determined Component: multiprecision
Version: Boost 1.57.0 Severity: Problem
Keywords: Cc:

Description

At line 508:

   *result.limbs() = boost::math::gcd(*a.limbs(), *b.limbs());

This is in a template, so MSVC has no problems with it, only g++ and clang++ complain.

The reason this was not a problem before was that boost/multiprecision/rational_adaptor.hpp uses Rational, which used to include math/common_factor_rt.hpp, but no longer does.

Either math:: should be made integer::, the include added, or both.

Change History (1)

comment:1 by John Maddock, 8 years ago

Resolution: fixed
Status: newclosed

This should be fixed in develop now.

Note: See TracTickets for help on using tickets.