Make constructors for BitMask constexpr. (#31343)

BitFlags already has these constexpr, it seems we likely want the same from bit masks.
project-chip · Jan 13, 2024 · 1057336 · 1057336
1 parent 7742714
commit 1057336
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/lib/support/BitMask.h b/src/lib/support/BitMask.h
@@ -33,20 +33,20 @@ class BitMask : public BitFlags<FlagsEnum, StorageType>
 public:
     using IntegerType = typename BitFlags<FlagsEnum, StorageType>::IntegerType;
 
-    BitMask() : BitFlags<FlagsEnum, StorageType>() {}
-    BitMask(const BitFlags<FlagsEnum, StorageType> & other) : BitFlags<FlagsEnum, StorageType>(other) {}
-    BitMask(BitFlags<FlagsEnum, StorageType> && other) : BitFlags<FlagsEnum, StorageType>(std::move(other)) {}
+    constexpr BitMask() : BitFlags<FlagsEnum, StorageType>() {}
+    constexpr BitMask(const BitFlags<FlagsEnum, StorageType> & other) : BitFlags<FlagsEnum, StorageType>(other) {}
+    constexpr BitMask(BitFlags<FlagsEnum, StorageType> && other) : BitFlags<FlagsEnum, StorageType>(std::move(other)) {}
     BitMask(const BitMask &) = default;
 
     explicit BitMask(FlagsEnum value) : BitFlags<FlagsEnum, StorageType>(value) {}
     explicit BitMask(IntegerType value) : BitFlags<FlagsEnum, StorageType>(value) {}
 
     template <typename... Args>
-    BitMask(FlagsEnum flag, Args &&... args) : BitFlags<FlagsEnum, StorageType>(flag, std::forward<Args>(args)...)
+    constexpr BitMask(FlagsEnum flag, Args &&... args) : BitFlags<FlagsEnum, StorageType>(flag, std::forward<Args>(args)...)
     {}
 
     template <typename... Args>
-    BitMask(IntegerType value, Args &&... args) : BitFlags<FlagsEnum, StorageType>(value, std::forward<Args>(args)...)
+    constexpr BitMask(IntegerType value, Args &&... args) : BitFlags<FlagsEnum, StorageType>(value, std::forward<Args>(args)...)
     {}
 
     BitMask & operator=(const BitMask &) = default;