boostorg / proto

Boost.org proto module
http://boost.org/libs/proto
21 stars 38 forks source link

Compiler bug was fixed in VC++ 2013, workaround no longer necessary. #2

Closed dodheim closed 5 years ago

dodheim commented 9 years ago

This bug was fixed in VC++ 2013, the workaround is no longer necessary. I noticed the fix as of VC++ 2013 Update 2, so I don't know if the fix was in RTM or in an update, so it may be more prudent to set the value to 1900 instead...

ericniebler commented 9 years ago

so it may be more prudent to set the value to 1900 instead

I think that's wise, unless we can confirm that the fix was in RTM.

ericniebler commented 9 years ago

Do you want to change the value to 1900 or should I?

QuellaZhang commented 5 years ago

Hi, @dodheim @ericniebler

Thanks for the workaround for MSVC, but the MSVC recently emitted error due to this workaround. And we verified if removed it Boost/proto will build successfully. Can you merge this PR to Boost master branch to remove the workaround? Thank you very much.