Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix macro redefinition warning in VS2015 #22

Closed
wants to merge 1 commit into from

Conversation

mauve
Copy link

@mauve mauve commented Aug 20, 2015

@mauve
Copy link
Author

mauve commented Sep 2, 2015

ping?

@a-teammate
Copy link

i did review this code as well and it works and looks good.
However i do not quite get the need for the check for BOOST_VERSION here.. but maybe im missing some background at this point yet..

@MarcelRaad
Copy link

@a-teammate There is also a standalone version of Asio not bundled with Boost, which needs the Boost version check. It already has this fix:
chriskohlhoff/asio@15803ab

@rustyx
Copy link

rustyx commented Feb 5, 2016

Hello? Anyone out there listening??!
Takes a fricking year to get a one line fix into Boost...

@MarcelRaad
Copy link

It's already in Boost 1.60: b9cb614

@burannah
Copy link

Can we have this change also in non-Boost Asio?

@chriskohlhoff
Copy link
Collaborator

Already fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants