TheKucks/eden
1
0
Fork 0
forked from eden-emu/eden
Code Pull requests Activity

common/bit_field: Silence sign-conversion warnings

Browse source 
We can just use numeric_limits instead of relying on wraparound behavior
here.
This commit is contained in:
Lioncash 2019-11-13 14:41:24 -05:00
parent 87039f537f
commit 95ed7df791
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
src/common/bit_field.h
View file

@ -135,7 +135,8 @@ public:
/// Constants to allow limited introspection of fields if needed
static constexpr std::size_t position = Position;
static constexpr std::size_t bits = Bits;
static constexpr StorageType mask = (((StorageType)~0) >> (8 * sizeof(T) - bits)) << position;
static constexpr StorageType mask = StorageType(
(std::numeric_limits<StorageType>::max() >> (8 * sizeof(T) - bits)) << position);
/**
* Formats a value by masking and shifting it according to the field parameters. A value
@ -143,7 +144,7 @@ public:
* the results together.
*/
static constexpr FORCE_INLINE StorageType FormatValue(const T& value) {
return ((StorageType)value << position) & mask;
return (static_cast<StorageType>(value) << position) & mask;
}
/**