Boost logo

Boost :

Subject: Re: [boost] [1.47.0] Release branch now closed
From: Anthony Williams (anthony.ajw_at_[hidden])
Date: 2011-06-17 12:53:17


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

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

Anthony

-- 
Author of C++ Concurrency in Action     http://www.stdthread.co.uk/book/
just::thread C++0x thread library             http://www.stdthread.co.uk
Just Software Solutions Ltd       http://www.justsoftwaresolutions.co.uk
15 Carrallack Mews, St Just, Cornwall, TR19 7UL, UK. Company No. 5478976

Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk