Re: [boost] [integer][math]Heads up on revised gcd/lcm code

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 
failures.

> 
> All have tentative fixes in place which I think will fix things, but I'm 
> waiting on the tests to cycle.
> 
> John