Clean up the fmt macro and format the tests

Author: rick-de-water

library/core/src/num/nonzero.rs

@@ -110,28 +110,40 @@ impl_zeroable_primitive!(
 pub struct NonZero<T: ZeroablePrimitive>(T::NonZeroInner);
 
 macro_rules! impl_nonzero_fmt {
-    ($Trait:ident, $Feature:literal, $Since:literal) => {
-        #[stable(feature = $Feature, since = $Since)]
-        impl<T> fmt::$Trait for NonZero<T>
-        where
-            T: ZeroablePrimitive + fmt::$Trait,
-        {
-            #[inline]
-            fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
-                self.get().fmt(f)
+    ($(#[$Attribute:meta] $Trait:ident)*) => {
+        $(
+            #[$Attribute]
+            impl<T> fmt::$Trait for NonZero<T>
+            where
+                T: ZeroablePrimitive + fmt::$Trait,
+            {
+                #[inline]
+                fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
+                    self.get().fmt(f)
+                }
             }
-        }
+        )*
     };
 }
 
-impl_nonzero_fmt!(Debug, "nonzero", "1.28.0");
-impl_nonzero_fmt!(Display, "nonzero", "1.28.0");
-impl_nonzero_fmt!(Binary, "nonzero", "1.28.0");
-impl_nonzero_fmt!(Octal, "nonzero", "1.28.0");
-impl_nonzero_fmt!(LowerHex, "nonzero", "1.28.0");
-impl_nonzero_fmt!(UpperHex, "nonzero", "1.28.0");
-impl_nonzero_fmt!(LowerExp, "nonzero_fmt_exp", "CURRENT_RUSTC_VERSION");
-impl_nonzero_fmt!(UpperExp, "nonzero_fmt_exp", "CURRENT_RUSTC_VERSION");
+impl_nonzero_fmt! {
+    #[stable(feature = "nonzero", since = "1.28.0")]
+    Debug
+    #[stable(feature = "nonzero", since = "1.28.0")]
+    Display
+    #[stable(feature = "nonzero", since = "1.28.0")]
+    Binary
+    #[stable(feature = "nonzero", since = "1.28.0")]
+    Octal
+    #[stable(feature = "nonzero", since = "1.28.0")]
+    LowerHex
+    #[stable(feature = "nonzero", since = "1.28.0")]
+    UpperHex
+    #[stable(feature = "nonzero_fmt_exp", since = "CURRENT_RUSTC_VERSION")]
+    LowerExp
+    #[stable(feature = "nonzero_fmt_exp", since = "CURRENT_RUSTC_VERSION")]
+    UpperExp
+}
 
 macro_rules! impl_nonzero_auto_trait {
     (unsafe $Trait:ident) => {

library/core/tests/nonzero.rs

@@ -358,7 +358,10 @@ fn test_signed_nonzero_neg() {
 #[test]
 fn test_nonzero_fmt() {
     let i = format!("{0}, {0:?}, {0:x}, {0:X}, {0:#x}, {0:#X}, {0:o}, {0:b}, {0:e}, {0:E}", 42);
-    let nz = format!("{0}, {0:?}, {0:x}, {0:X}, {0:#x}, {0:#X}, {0:o}, {0:b}, {0:e}, {0:E}", NonZero::new(42).unwrap());
+    let nz = format!(
+        "{0}, {0:?}, {0:x}, {0:X}, {0:#x}, {0:#X}, {0:o}, {0:b}, {0:e}, {0:E}",
+        NonZero::new(42).unwrap()
+    );
 
     assert_eq!(i, nz);
-}
+}