boostorg / pool

Boost.org pool module
http://boost.org/libs/pool
Boost Software License 1.0
49 stars 50 forks source link

improve guards around boost/thread/mutex.hpp #11

Closed nschloe closed 9 years ago

nschloe commented 9 years ago

Fixes bug #7085 (https://svn.boost.org/trac/boost/ticket/7085).

mclow commented 9 years ago

So, the goal here is if you define BOOST_POOL_NO_MT, then you don't include "mutex.hpp"?

nschloe commented 9 years ago

Indeed. The discussion at https://svn.boost.org/trac/boost/ticket/7085 gives more details.

mclow commented 9 years ago

Looks good to me. Please watch the test bots and ping me in a few days to merge to master (after the 1.59.0 release is done).

nschloe commented 9 years ago

Um, wait. Am I supposed to watch the test bots? Okay. Where can I do that?

mclow commented 9 years ago

Here: http://www.boost.org/development/tests/develop/developer/pool.html

nschloe commented 9 years ago

ping me in a few days to merge to master

@mclow Ping!

nschloe commented 9 years ago

@mclow Has this been merged to master yet?

mclow commented 9 years ago

Just did.

nschloe commented 9 years ago

Perfect, thanks!