On 4/25/2017 3:22 AM, John Maddock via Boost wrote:
On 24/04/2017 20:03, Edward Diener via Boost wrote:
On 4/24/2017 1:37 PM, John Maddock via Boost wrote:
First some history....
Some time ago, the old gcd/lcm code was moved from Boost.Math to Boost.Integer as a means of reducing inter-library dependencies. Unfortunately a number of tasks were never completed: the docs weren't added to Boost.Integer, and the Boost.Math code never got updated to redirect to the moved headers. Fast forward a couple of years and when Jeremy Murphy contributed a new version of the gcd/lcm code as part of supporting polynomial gcd, and since I'd completely forgotten about the move, this got integrated into Boost.Math leaving us with two divergent versions.
As part of sorting this mess out, I've now pushed to Boost.Integer a "best of both" version of this code, so far as I can tell, all dependent libraries are unaffected by the change, though I do need to do some more work on supporting obsolete compilers I no longer have access to.
To which obsolete compilers are you referring ?
Primarily VC7.1, 8, 9. Also Sun+STLPort (which I do have access to, but I haven't fired up that virtual machine yet).
VC 8,9 all pass both integer and rational which uses integer for gcd/lcm. Sun+STLPort can still be any Sun/Oracle release I believe, but I will test against 12.2 through 12.5 and let you know if I see any failures.
All have tentative fixes in place which I think will fix things, but I'm waiting on the tests to cycle.
John