]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
rust: str: simplify KUnit tests `format!` macro
authorMiguel Ojeda <ojeda@kernel.org>
Mon, 26 May 2025 18:01:42 +0000 (20:01 +0200)
committerMiguel Ojeda <ojeda@kernel.org>
Tue, 27 May 2025 18:09:59 +0000 (20:09 +0200)
Simplify the `format!` macro used in the tests by using
`CString::try_from_fmt` and directly `unwrap()`ing.

This will allow us to change both `unwrap()`s here in order to showcase
the `?` operator support now that the tests are KUnit ones.

Reviewed-by: David Gow <davidgow@google.com>
Acked-by: Danilo Krummrich <dakr@kernel.org>
[ Split from the next commit as suggested by Tamir. - Miguel ]
Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
rust/kernel/str.rs

index 9b5228025e2c6cc88575583ae568a6b69ad3510f..52a500742c1a75e4d108fa00ca9dcc246a64296e 100644 (file)
@@ -576,25 +576,9 @@ macro_rules! c_str {
 mod tests {
     use super::*;
 
-    struct String(CString);
-
-    impl String {
-        fn from_fmt(args: fmt::Arguments<'_>) -> Self {
-            String(CString::try_from_fmt(args).unwrap())
-        }
-    }
-
-    impl Deref for String {
-        type Target = str;
-
-        fn deref(&self) -> &str {
-            self.0.to_str().unwrap()
-        }
-    }
-
     macro_rules! format {
         ($($f:tt)*) => ({
-            &*String::from_fmt(::kernel::fmt!($($f)*))
+            CString::try_from_fmt(::kernel::fmt!($($f)*)).unwrap().to_str().unwrap()
         })
     }