From: David Abrahams (dave_at_[hidden])
Date: 2004-05-18 10:08:49
"John Maddock" <john_at_[hidden]> writes:
>> I know that, and I don't mean to sound harsh, but really that's no
>> excuse. Before stuff like this is checked in, it needs to be tested,
>> since it affects all boost developers using that compiler.
> It *was* tested with Intel 7 and Intel 8, and does exactly what it's
> supposed to do with those compilers. Intel 6 wasn't tested because
> I don't have access to that compiler
> - it's not even supported by Intel any more is it?
Not sure why that matters. People still use it.
> BTW the offending code:
> #if BOOST_INTEL_CXX_VERSION < 700
> # define BOOST_NO_INTRINSIC_WCHAR_T
> Has been in cvs for a long time, it was checked in with:
> Revision 1.23 - (download), view (text) (markup) (annotate) - [select for
> Sun Jun 1 18:06:27 2003 UTC (11 months, 2 weeks ago) by beman_dawes
> Changes since 1.22: +1 -1 lines
> Diff to previous 1.22
> BOOST_NO_INTRINSIC_WCHAR_T now correct for 7.0 on Win32, based on
> config_info report
> The only difference is that the code now checks to verify that it is
> correct - in other words we're detecting an error that previously went
> unnoticed unless you tried including type_traits/*.hpp or something.
I don't think even that would've detected the issue, since before this
I was using my Intel6 toolset smoothly with all kinds of Boost code,
especially type traits.
-- Dave Abrahams Boost Consulting http://www.boost-consulting.com
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk