Re: [boost] [1.47.0] Release branch now closed

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:
>>
>> tss_pte.cpp#14
>> 	#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
> tests.
>
> 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).

OK.

-- 
-- 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