|
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