diff options
author | Miguel Ojeda <ojeda@kernel.org> | 2025-05-26 20:01:42 +0200 |
---|---|---|
committer | Miguel Ojeda <ojeda@kernel.org> | 2025-05-27 20:09:59 +0200 |
commit | 1486554392e242da5cbe95092d8dfec887bb8cca (patch) | |
tree | 62548be013936e9dfac2cb97577d0c49b34d673b /rust/kernel/str.rs | |
parent | 028df914e5466a02326829427bb8e26a31a05545 (diff) |
rust: str: simplify KUnit tests `format!` macro
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>
Diffstat (limited to 'rust/kernel/str.rs')
-rw-r--r-- | rust/kernel/str.rs | 18 |
1 files changed, 1 insertions, 17 deletions
diff --git a/rust/kernel/str.rs b/rust/kernel/str.rs index 9b5228025e2c..52a500742c1a 100644 --- a/rust/kernel/str.rs +++ b/rust/kernel/str.rs @@ -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() }) } |