Cleanup superflous Boost headers - #164
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## develop #164 +/- ##
========================================
Coverage 95.79% 95.79%
========================================
Files 117 117
Lines 7660 7660
========================================
Hits 7338 7338
Misses 322 322
Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|
mborland
left a comment
There was a problem hiding this comment.
Thanks! I need to look more closely at the rest of the dependencies in this lib. I'm sure almost all can go away.
|
Given the amount of C++11 features already required I guess that allows dropping more, e.g. Boost.bind Note that this library is still very valuable: C++11 does not define how distributions work (e.g. how many calls to the RNG occur) and how distributions and generators are serialized. |
One of the dependencies is Math so we're at least up to C++14 here.
For sure. We also have newer components with better properties than those offered by |
In most cases
std::uint32_tetc and in all casesstd::numeric_limitsis used.So include the standard headers and replace the remaining Boost type usages