[boost] [Boost-bugs] [ boost-Patches-1489352 ] evc4 port: Interlocked*-functions without volatile

From: SourceForge.net (noreply_at_[hidden])
Date: 2006-05-30 06:47:33

Patches item #1489352, was opened at 2006-05-16 10:20
Message generated for change (Settings changed) made by pdimov
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=307586&aid=1489352&group_id=7586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Pending
Resolution: Accepted
Priority: 5
Submitted By: Michael Fink (vividos)
Assigned to: Peter Dimov (pdimov)
Summary: evc4 port: Interlocked*-functions without volatile

Initial Comment:
This patch fixes the definition of the
Interlocked*-functions when using Windows CE
(_WIN32_WCE is defined). The functions use the "old"
(VC6 without platform SDK) forms of the functions
without the volatile keyword. The patch can be applied
to both CVS HEAD and the 1_34 branch, since the file
wasn't modified since then.

----------------------------------------------------------------------

Comment By: Peter Dimov (pdimov)
Date: 2006-05-18 01:41

Message:
Logged In: YES 
user_id=305912

Applied to 1.34 and HEAD.

----------------------------------------------------------------------

Comment By: Michael Fink (vividos)
Date: 2006-05-17 10:57

Message:
Logged In: YES 
user_id=124874

I added a new version of the patch that avoids WINAPI
(replaced by __cdecl, as used by the BOOST_MSVC part of the
file. Including <windows.h> is not needed here, I think. In
evc4 there are no intrinsics for the Interlocked-functions
and the pragmas produce errors telling so. The _WIN32_WCE
part of the file provides the BOOST_INTERLOCKED_ macros in
its own section now.

----------------------------------------------------------------------

Comment By: Peter Dimov (pdimov)
Date: 2006-05-16 13:42

Message:
Logged In: YES 
user_id=305912

The patch doesn't look correct to me. Using WINAPI would
require inclusion of <windows.h>; in this case just defining
BOOST_USE_WINDOWS_H would be a better approach.

It also patches the intrinsic section without also declaring
the functions as intrinsics; is this intended? Does evc4
have intrinsic support for these functions? If it doesn't,
we need to disallow the intrinsic path for _WIN32_WCE and
patch the WIN32 section instead.

----------------------------------------------------------------------

Comment By: Michael Fink (vividos)
Date: 2006-05-16 10:40

Message:
Logged In: YES 
user_id=124874

Changed summary: it's only evc4 of course, since evc3 is too
non-standard to support boost.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=307586&aid=1489352&group_id=7586

_______________________________________________
Boost-bugs mailing list
Boost-bugs_at_[hidden]
https://lists.sourceforge.net/lists/listinfo/boost-bugs