Subject: Re: [boost] [1.47.0] Release branch now closed
From: Rene Rivera (grafikrobot_at_[hidden])
Date: 2011-06-17 14:18:31
On 6/17/2011 11:53 AM, Anthony Williams wrote:
> Rene Rivera<grafikrobot_at_[hidden]> writes:
>> I mean this compile problem:
>> #if #if (defined(__MINGW32__)&& !defined(_WIN64)) || defined(__MINGW64__)
>> I.e. the double #if.
> That's not present in the actual code (which is already merged to
> release anyway).
OK, so now I'm confused.. Could you please post a patch of what the
change to the release branch is? From what I understand now is that
neither the changeset nor the current state of threads files is telling
me what the merge is.
>> And, at least a potential for problems depending on which CPP hits this:
>> extern "C"BOOST_INTERLOCKED_IMPORT long __stdcall
>> InterlockedIncrement( long volatile * );
>> That is, the lack of space between "C" and BOOST_INTERLOCKED_IMPORT.
> OK, that's really there in the file, and this is the file that hasn't
> yet been merged, but it isn't causing a problem on the trunk regression
> I don't think the whitespace is important anyway --- the grammar is:
> extern /string-literal/ /declaration/
> Neither /string-literal/ nor /declaration/ require leading or trailing
> whitespace, so it should be OK (and appears to be in practice for the
> compilers tested on trunk).
-- -- Grafik - Don't Assume Anything -- Redshift Software, Inc. - http://redshift-software.com -- rrivera/acm.org (msn) - grafik/redshift-software.com -- 102708583/icq - grafikrobot/aim,yahoo,skype,efnet,gmail
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk