From c71b23d81d31c2474ee92c8e1b7cc6d7b9a3785f Mon Sep 17 00:00:00 2001 From: "Tobin C. Harding" Date: Mon, 2 Sep 2024 08:20:52 +1000 Subject: [PATCH] Remove bool from cont_assert It was correctly pointed out during review of #3215 (when we made `const_assert` panic) that using a `bool` added no additional information. Remove the `bool` and just use unit. --- internals/src/macros.rs | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/internals/src/macros.rs b/internals/src/macros.rs index 0514d7c4a..a12802e8d 100644 --- a/internals/src/macros.rs +++ b/internals/src/macros.rs @@ -140,12 +140,11 @@ macro_rules! debug_from_display { #[macro_export] macro_rules! const_assert { ($x:expr $(; $message:expr)?) => { - const _: bool = { + const _: () = { if !$x { // We can't use formatting in const, only concating literals. panic!(concat!("assertion ", stringify!($x), " failed" $(, ": ", $message)?)) } - $x }; } }