Make a few palette::cast::* functions const fn

Ogeon · Jul 31, 2024 · 13368ac · 13368ac
1 parent b60a87d
commit 13368ac
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 18 deletions.
diff --git a/palette/src/cast/array.rs b/palette/src/cast/array.rs
@@ -238,15 +238,12 @@ where
 /// let array3 = <&[_; 3]>::from(&color);
 /// ```
 #[inline]
-pub fn into_array_ref<T>(value: &T) -> &T::Array
+pub const fn into_array_ref<T>(value: &T) -> &T::Array
 where
     T: ArrayCast,
 {
-    assert_eq!(core::mem::size_of::<T::Array>(), core::mem::size_of::<T>());
-    assert_eq!(
-        core::mem::align_of::<T::Array>(),
-        core::mem::align_of::<T>()
-    );
+    assert!(core::mem::size_of::<T::Array>() == core::mem::size_of::<T>());
+    assert!(core::mem::align_of::<T::Array>() == core::mem::align_of::<T>());
 
     let value: *const T = value;
 
@@ -282,15 +279,12 @@ where
 /// let color3 = <&Srgb<u8>>::from(&array);
 /// ```
 #[inline]
-pub fn from_array_ref<T>(value: &T::Array) -> &T
+pub const fn from_array_ref<T>(value: &T::Array) -> &T
 where
     T: ArrayCast,
 {
-    assert_eq!(core::mem::size_of::<T::Array>(), core::mem::size_of::<T>());
-    assert_eq!(
-        core::mem::align_of::<T::Array>(),
-        core::mem::align_of::<T>()
-    );
+    assert!(core::mem::size_of::<T::Array>() == core::mem::size_of::<T>());
+    assert!(core::mem::align_of::<T::Array>() == core::mem::align_of::<T>());
 
     let value: *const T::Array = value;
 

diff --git a/palette/src/cast/uint.rs b/palette/src/cast/uint.rs
@@ -110,12 +110,12 @@ where
 /// assert_eq!(cast::into_uint_ref(&color), &0xFF17C64C);
 /// ```
 #[inline]
-pub fn into_uint_ref<T>(value: &T) -> &T::Uint
+pub const fn into_uint_ref<T>(value: &T) -> &T::Uint
 where
     T: UintCast,
 {
-    assert_eq!(core::mem::size_of::<T::Uint>(), core::mem::size_of::<T>());
-    assert_eq!(core::mem::align_of::<T::Uint>(), core::mem::align_of::<T>());
+    assert!(core::mem::size_of::<T::Uint>() == core::mem::size_of::<T>());
+    assert!(core::mem::align_of::<T::Uint>() == core::mem::align_of::<T>());
 
     let value: *const T = value;
 
@@ -133,12 +133,12 @@ where
 /// assert_eq!(cast::from_uint_ref::<PackedArgb>(&0xFF17C64C), &color);
 /// ```
 #[inline]
-pub fn from_uint_ref<T>(value: &T::Uint) -> &T
+pub const fn from_uint_ref<T>(value: &T::Uint) -> &T
 where
     T: UintCast,
 {
-    assert_eq!(core::mem::size_of::<T::Uint>(), core::mem::size_of::<T>());
-    assert_eq!(core::mem::align_of::<T::Uint>(), core::mem::align_of::<T>());
+    assert!(core::mem::size_of::<T::Uint>() == core::mem::size_of::<T>());
+    assert!(core::mem::align_of::<T::Uint>() == core::mem::align_of::<T>());
 
     let value: *const T::Uint = value;