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

Replace BOOST_SYSTEM_NO_DEPRECATED with BOOST_SYSTEM_ENABLE_DEPRECATED #18

Merged
merged 1 commit into from
Oct 22, 2017

Conversation

pdimov
Copy link
Member

@pdimov pdimov commented Oct 21, 2017

No description provided.

@pdimov
Copy link
Member Author

pdimov commented Oct 22, 2017

Beman, can you please take a look at this and decide whether we want it in 1.66, as there are just a few days left.

@Beman
Copy link
Contributor

Beman commented Oct 22, 2017 via email

@pdimov
Copy link
Member Author

pdimov commented Oct 22, 2017

OK, let's see how it goes.

@pdimov pdimov merged commit 41e0404 into develop Oct 22, 2017
@pdimov
Copy link
Member Author

pdimov commented Oct 22, 2017

As a next step, I think that we need to consider routing boost::error_code and friends to std::error_code et al, if the compiler supports the latter. This can also potentially break code, of course. But "use std::error_code if it exists, otherwise boost::error_code" is something many libraries already do, and due to error_code being an interface type, this creates incompatibilities between libraries that choose one over the other.

@ned14
Copy link
Member

ned14 commented Oct 22, 2017

The patchset looked good.

I agree with the idea that error_code, error_category etc become mapped to the std editions when on C++ 11 or better. It allows all the header only libraries currently incorrectly using header only Boost.System to become correct if compiled on C++ 11 or better.

One might consider announcing in the release notes that the next Boost will be the last to use the local implementation in C++ 11 or better.

Also, one might consider also rerouting boost::current_exception() to std::current_exception(). The latter is surely always going to be equal or better to the Boost implementation.

@pdimov
Copy link
Member Author

pdimov commented Oct 22, 2017

This last suggestion would be better placed in https://github.com/boostorg/exception. It didn't have issues enabled, but it does now. :-)

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.

None yet

3 participants