-
Notifications
You must be signed in to change notification settings - Fork 183
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
Bit Mask utility function #3265
Conversation
Codecov Report
@@ Coverage Diff @@
## python #3265 +/- ##
=======================================
+ Coverage 86% 86% +<1%
=======================================
Files 530 533 +3
Lines 25647 25643 -4
=======================================
+ Hits 22061 22062 +1
+ Misses 3586 3581 -5
Continue to review full report at Codecov.
|
src/utils/include/utils/mask.hpp
Outdated
template <class T, class Integral> | ||
auto mask(Integral mask, T t) | ||
-> std::enable_if_t<std::is_unsigned<Integral>::value && | ||
(8 * sizeof(Integral) >= tuple_size<T>::value), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where does the 8 come from?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bits vs bytes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I gave it a name, you were the second person to ask this ^^
bors r=RudolfWeeber |
3265: Bit Mask utility function r=RudolfWeeber a=fweik Description of changes: - tuple interface for Utils::{Array, Vector} - Extract bit masking utility function to utils - Some small code modernization Co-authored-by: Florian Weik <[email protected]>
Build failed |
bors retry |
3265: Bit Mask utility function r=RudolfWeeber a=fweik Description of changes: - tuple interface for Utils::{Array, Vector} - Extract bit masking utility function to utils - Some small code modernization Co-authored-by: Florian Weik <[email protected]>
Build succeeded |
Description of changes: