|
Boost : |
From: David Abrahams (dave_at_[hidden])
Date: 2003-08-18 15:08:23
Jens Maurer <Jens.Maurer_at_[hidden]> writes:
> Fernando Cacciola wrote:
>> Recently, Jens Maurer changed the guard at function scope
>> from:
>>
>> #ifndef __GNUC__
>> to
>> #ifndef BOOST_NO_STDC_NAMESPACE
>>
>> and honestly, I didn't looked much at it as I should.
>>
>> BOOST_NO_STDC_NAMESPACE is documented to relate to C names,
>> but swap is a C++ name so I don't think such macro
>> should be used here.
>
> The CVS change of optional.hpp:1.10 is definitely incorrect,
> because STDC_NAMESPACE refers to C names, not C++ names.
> Sorry.
>
> However, just reverting the patch will make gcc-3.3
> non-functional, because std::swap(int,int) (for example)
> is not going to be found.
>
> I've checked in a better fix to the main branch. optional_test.cpp
> now works with gcc 2.95, gcc 3.0 and gcc 3.3 on Linux.
> Please test on other platforms and (optionally) transport
> the fix to the 1.30.0 CVS branch.
There is no chance of that happening before 1.30.2 is released (and
fact if anybody touches RC_1_30_0 before 1.30.2 goes out there'll be
heck to pay). I believe http://tinyurl.com/ke9z shows that GCC 3.3.1
is healthy now, though.
-- Dave Abrahams Boost Consulting 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