Patches item #1627420, was opened at 2007-01-03 18:17 Message generated for change (Settings changed) made by david_abrahams You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=307586&aid=1627420&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: Open Resolution: None Priority: 5 Private: No Submitted By: sohail (sohail_)

Assigned to: Anthony Williams (anthonyw) Summary: [thread] MSVS: Allow use of thread headers with /Za

Initial Comment: The following code will not compile if built with /Za on Visual C++ 7.1+: #include <boost/thread/once.hpp> int main(){} It will fail with the following assertion: # error "Threading support unavaliable: it has been explicitly disabled with BOOST_DISABLE_THREADS" The reason is that the Boost.Config machinery realizes that you cannot use the Windows API unless you enable language extensions. As a result, threads cannot be available. This is correct behaviour when you are indeed *using* the Windows API. However, in the above case where you separately compile the threads library, you do not actually use the Windows API => language extensions are not necessary => no need to explicitly specify BOOST_DISABLE_WIN32. I therefore submit that there be some machinery/configuration variables to allow libraries to specify whether or not they require the Windows API. The proposed patch does not modify existing behaviour. That is, if the config machinery decides that the API is needed, it is required, unless explicitly disabled. The code to do this in library xyz would look something like this: #if defined(BOOST_BUILDING_WHATEVER) # define BOOST_NO_REQUIRE_WIN32 #endif #include <boost/config.hpp> I am uncertain of the exact name of the macro that is required but I am certain a macro is required. The attached patch is against 1.33.1. I don't think there is any rush to put this into 1.34. I'm pretty sure it would be nice, but people can manually patch it until 1.35 whenever that is released. Thanks, Sohail ---------------------------------------------------------------------- Comment By: sohail (sohail_) Date: 2007-01-05 22:46 Message: Logged In: YES user_id=1682817 Originator: YES File Added: 0_boost_building_threads.patch ---------------------------------------------------------------------- Comment By: sohail (sohail_) Date: 2007-01-05 15:23 Message: Logged In: YES user_id=1682817 Originator: YES See also this thread: http://thread.gmane.org/gmane.comp.lib.boost.user/23774/focus=23774 For a discussion that seems to converge on having Boost.Config not touch BOOST_DISABLE_THREADS ---------------------------------------------------------------------- Comment By: sohail (sohail_) Date: 2007-01-03 18:40 Message: Logged In: YES user_id=1682817 Originator: YES File Added: boost_no_require_win32.patch ---------------------------------------------------------------------- Comment By: sohail (sohail_) Date: 2007-01-03 18:25 Message: Logged In: YES user_id=1682817 Originator: YES That should be: #if !defined(BOOST_BUILDING_WHATEVER) :) ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=307586&aid=1627420&group_id=7586 ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Boost-bugs mailing list Boost-bugs@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/boost-bugs