Cleanup dead cxx11 guards from /include/boost/pending/container_traits.hpp - #588
Conversation
|
Boost dependency footprint vs Header-inclusion weights (graph files pulling each direct dependency in): No header-inclusion-weight changes. Transitive Boost modules: 65 → 65 (0) |
|
Compiler-warning counts vs
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! 🚀 New features to boost your workflow:
|
jeremy-murphy
left a comment
There was a problem hiding this comment.
Looks great, thank you! Just some minor quibble about unnecessary changes.
Becheler
left a comment
There was a problem hiding this comment.
Thank you so much for this first PR !
It revealed a consequent gap in the unit test coverage: I guess because historically the CI was running on old compilers, the modern paths you accidentally deleted were never tested ! So the PR breaks users but CI is green, what a wonderful world ! 😆
The solution is relatively straightforwards (delete the question #ifndef BOOST_NO_CXX11_HDR_UNORDERED_MAP but keep the overload it guards), I indicated it in comment, just let me know if you have questions 🤗
2af9df7 to
9c5a0ca
Compare
|
Hi @Becheler. I updated the PR, but I just noticed you assigned this PR to yourself. Are you wanting to take this one, perhaps to add test coverage? |
|
Also the CI run failed for |
|
@samkelemen you good, I just use assignment to track what are my tasks in the project's Kanban, I added you too. It's mostly cosmetic and for tracking, it does not signal ownership 😄 Also you will learn not to put too much attention in the drone job, it's runs some complex stuff related to the general boost ecosystem and when it fails it's most of the time not related directly to your changes. |
|
@samkelemen congratulations on your first merged Boost PR ! 🥳 We love that you made it as a new contributor ! Your profile should show up in the next review report 🎉 Thank you for your help 🤗 |
Before submitting
developbranch.Type of change
Does this PR introduce a breaking change?
What this PR does
This PR removes the following guards from
include/boost/pending/container_traits.hpp:#ifndef BOOST_NO_CXX11_HDR_UNORDERED_SET#ifndef BOOST_NO_CXX11_HDR_UNORDERED_MAP#ifdef BOOST_NO_CXX11_RVALUE_REFERENCESMotivation
Fixes #587
Checklist
b2in thetest/directory).