Subject: Re: [boost] [chrono] patch for VC9 duration conversions
From: David Deakins (ddeakins_at_[hidden])
Date: 2011-02-23 15:38:15
I had actually only looked at the VC9 issues that were directly causing
test failures for the chrono test suite, so I hadn't noticed the other
VC9-related ifdefs. However, I went through those files and removed all
the ifdefs and I do not see any failures or compiler crashes. Is it
possible that refactoring elsewhere in the code has made most of these
fixes no longer necessary?
Attached is a broader patch with my initial changes plus:
- is_evenly_divisible_by is now moved from ratio_detail to a
chrono_detail in a new file chrono/detail/is_evenly_divisible_by.hpp
- the other VC9-related ifdefs have been removed.
Let me know if you think this is good to apply to the trunk.
On 2/23/2011 5:40 AM, Vicente Botet wrote:
> thanks for looking at this VC9 issue. And congratulation for the first
> Chrono ticket I'm aware off :)
> I was unable to find out why VC9 had some problems with these enable_if
> I see that you have removed 2 cases, but there are others. Have you tried to
> analyze them?
> The patch seem good to me except that we are adding a ratio detail. Maybe it
> will be better to add the is_evenly_divisible_by metafunction in the details
> of Boost.Chrono for the moment.
> Please be free to apply the patch yourself. I will try to see if similar
> workarounds can solve the other cases.
> Many thanks Dave,
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk