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

Add noexcept support to the library #59

Closed
impugachev opened this issue Mar 9, 2021 · 5 comments
Closed

Add noexcept support to the library #59

impugachev opened this issue Mar 9, 2021 · 5 comments

Comments

@impugachev
Copy link

Why doesn't the library use noexcept anywhere? I work in an environment with no exceptions, so intrusive containers are exactly what I need, but when using Boost.Intrusive, there are some problems due to the complete lack of noexcept.

@igaztanaga
Copy link
Member

The reason is that Intrusive was written long before noexcept was available. Any hint on which operations are most needed to be noexcept?

@impugachev
Copy link
Author

First of all, hook constructors. Because of them, when inheriting, you have to explicitly write an empty noexcept constructor.

@igaztanaga
Copy link
Member

Thanks for the feedback. I plan to implement firs noexcept support for the next release.

@igaztanaga igaztanaga changed the title noexcept Add noexcept support to the library Apr 19, 2021
@igaztanaga
Copy link
Member

Thanks for the report. Let me know if the support is enough or I missed something important.

@impugachev
Copy link
Author

Many thanks! It looks like everything I need is here. I'll write if I find anything else while using it.

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

No branches or pull requests

2 participants