diff options
| author | Jacques Lucke <jacques@blender.org> | 2021-06-28 14:10:48 +0300 |
|---|---|---|
| committer | Jacques Lucke <jacques@blender.org> | 2021-06-28 14:16:32 +0300 |
| commit | f7e2559fd649610881b1749b6d30cc2ba9fcbdb6 (patch) | |
| tree | 07b9c37360596813abcd854eff4c6ca1b7359ae4 /source | |
| parent | 2dbb492268c192f617b812b1cec569eae8a1aed3 (diff) | |
BLI: improve enum operators
* Use unsigned integer types for bit operations.
* Use 64 bit integer instead of 32 bit.
* Support scoped enumes (aka `enum class`) by adding some more casts.
Diffstat (limited to 'source')
| -rw-r--r-- | source/blender/blenlib/BLI_utildefines.h | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/source/blender/blenlib/BLI_utildefines.h b/source/blender/blenlib/BLI_utildefines.h index 0ddabcaa2fb..5b84e050f82 100644 --- a/source/blender/blenlib/BLI_utildefines.h +++ b/source/blender/blenlib/BLI_utildefines.h @@ -788,23 +788,24 @@ extern bool BLI_memory_is_zero(const void *arr, const size_t arr_size); extern "C++" { \ inline constexpr _enum_type operator|(_enum_type a, _enum_type b) \ { \ - return static_cast<_enum_type>(static_cast<int>(a) | b); \ + return static_cast<_enum_type>(static_cast<uint64_t>(a) | static_cast<uint64_t>(b)); \ } \ inline constexpr _enum_type operator&(_enum_type a, _enum_type b) \ { \ - return static_cast<_enum_type>(static_cast<int>(a) & b); \ + return static_cast<_enum_type>(static_cast<uint64_t>(a) & static_cast<uint64_t>(b)); \ } \ inline constexpr _enum_type operator~(_enum_type a) \ { \ - return static_cast<_enum_type>(~static_cast<int>(a) & (2 * _max_enum_value - 1)); \ + return static_cast<_enum_type>(~static_cast<uint64_t>(a) & \ + (2 * static_cast<uint64_t>(_max_enum_value) - 1)); \ } \ inline _enum_type &operator|=(_enum_type &a, _enum_type b) \ { \ - return a = static_cast<_enum_type>(static_cast<int>(a) | b); \ + return a = static_cast<_enum_type>(static_cast<uint64_t>(a) | static_cast<uint64_t>(b)); \ } \ inline _enum_type &operator&=(_enum_type &a, _enum_type b) \ { \ - return a = static_cast<_enum_type>(static_cast<int>(a) & b); \ + return a = static_cast<_enum_type>(static_cast<uint64_t>(a) & static_cast<uint64_t>(b)); \ } \ } /* extern "C++" */ |