Subject: Re: [boost] [integer][math]Heads up on revised gcd/lcm code
From: Edward Diener (eldiener_at_[hidden])
Date: 2017-04-25 12:29:36
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
> All have tentative fixes in place which I think will fix things, but I'm
> waiting on the tests to cycle.
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk