On 6/17/2011 11:53 AM, Anthony Williams wrote:
Rene Rivera<grafikrobot@gmail.com> 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