REQUIRES macro should be in boost namespace

ptal / expected

What did you expect?

113 stars 18 forks source link

REQUIRES macro should be in boost namespace #82

Closed garyfurnish closed 9 years ago

garyfurnish commented 9 years ago

include/boost/expected/expected.hpp:# define REQUIRES(...) typename std::enable_if<VA_ARGS, void*>::type = 0 include/boost/expected/expected.hpp:# define T_REQUIRES(...) typename = typename std::enable_if<(VA_ARGS)>::ty

These should be prefixed by BOOST_ to prevent macro namespace contamination (in particular they interfere with the default clang REQUIRES macro for thread safety checking).

viboes commented 9 years ago

https://github.com/ptal/expected/commit/234fe47a3776b7c241473a5b9ea1dd9a52783d9a