Boost logo

Boost :

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


-- Grafik - Don't Assume Anything
-- Redshift Software, Inc. -
-- rrivera/ (msn) - grafik/
-- 102708583/icq - grafikrobot/aim,yahoo,skype,efnet,gmail

Boost list run by bdawes at, gregod at, cpdaniel at, john at