Remove panic optimization

With Rust 1.93 now stable, this brings the new implementation of
`format_args!`. Given this, the standalone method is no longer
necessary.

Author: jhpratt

time/src/duration.rs

@@ -15,8 +15,8 @@ use num_conv::prelude::*;
 #[expect(deprecated)]
 use crate::Instant;
 use crate::convert::*;
+use crate::error;
 use crate::internal_macros::const_try_opt;
-use crate::{error, panic};
 
 #[derive(Debug)]
 enum FloatConstructorError {
@@ -587,9 +587,11 @@ impl Duration {
     pub const fn seconds_f64(seconds: f64) -> Self {
         match Self::try_seconds_f64(seconds) {
             Ok(duration) => duration,
-            Err(FloatConstructorError::Nan) => panic("passed NaN to `time::Duration::seconds_f64`"),
+            Err(FloatConstructorError::Nan) => {
+                panic!("passed NaN to `time::Duration::seconds_f64`");
+            }
             Err(FloatConstructorError::NegOverflow | FloatConstructorError::PosOverflow) => {
-                panic("overflow constructing `time::Duration`")
+                panic!("overflow constructing `time::Duration`");
             }
         }
     }
@@ -606,9 +608,11 @@ impl Duration {
     pub const fn seconds_f32(seconds: f32) -> Self {
         match Self::try_seconds_f32(seconds) {
             Ok(duration) => duration,
-            Err(FloatConstructorError::Nan) => panic("passed NaN to `time::Duration::seconds_f32`"),
+            Err(FloatConstructorError::Nan) => {
+                panic!("passed NaN to `time::Duration::seconds_f32`");
+            }
             Err(FloatConstructorError::NegOverflow | FloatConstructorError::PosOverflow) => {
-                panic("overflow constructing `time::Duration`")
+                panic!("overflow constructing `time::Duration`");
             }
         }
     }
@@ -794,7 +798,7 @@ impl Duration {
         let nanoseconds = nanoseconds % Nanosecond::per_t::<i128>(Second);
 
         if seconds > i64::MAX as i128 || seconds < i64::MIN as i128 {
-            panic("overflow constructing `time::Duration`");
+            panic!("overflow constructing `time::Duration`");
         }
 
         // Safety: `nanoseconds` is guaranteed to be in range because of the modulus above.

time/src/format_description/well_known/iso8601/adt_hack.rs

@@ -5,7 +5,6 @@ use core::num::NonZero;
 #[cfg(feature = "formatting")]
 use super::Iso8601;
 use super::{Config, DateKind, FormattedComponents as FC, OffsetPrecision, TimePrecision};
-use crate::panic;
 
 // This provides a way to include `EncodedConfig` in documentation without displaying the type it is
 // aliased to.
@@ -107,23 +106,23 @@ impl Config {
             4 => FC::DateTime,
             5 => FC::DateTimeOffset,
             6 => FC::TimeOffset,
-            _ => panic("invalid configuration"),
+            _ => panic!("invalid configuration"),
         };
         let use_separators = match bytes[1] {
             0 => false,
             1 => true,
-            _ => panic("invalid configuration"),
+            _ => panic!("invalid configuration"),
         };
         let year_is_six_digits = match bytes[2] {
             0 => false,
             1 => true,
-            _ => panic("invalid configuration"),
+            _ => panic!("invalid configuration"),
         };
         let date_kind = match bytes[3] {
             0 => DateKind::Calendar,
             1 => DateKind::Week,
             2 => DateKind::Ordinal,
-            _ => panic("invalid configuration"),
+            _ => panic!("invalid configuration"),
         };
         let time_precision = match bytes[4] {
             0 => TimePrecision::Hour {
@@ -135,19 +134,19 @@ impl Config {
             2 => TimePrecision::Second {
                 decimal_digits: NonZero::new(bytes[5]),
             },
-            _ => panic("invalid configuration"),
+            _ => panic!("invalid configuration"),
         };
         let offset_precision = match bytes[6] {
             0 => OffsetPrecision::Hour,
             1 => OffsetPrecision::Minute,
-            _ => panic("invalid configuration"),
+            _ => panic!("invalid configuration"),
         };
 
         // No `for` loops in `const fn`.
         let mut idx = 7; // first unused byte
         while idx < EncodedConfig::BITS as usize / 8 {
             if bytes[idx] != 0 {
-                panic("invalid configuration");
+                panic!("invalid configuration");
             }
             idx += 1;
         }

time/src/internal_macros.rs

@@ -201,7 +201,7 @@ macro_rules! bug {
         compile_error!("provide an error message to help fix a possible bug")
     };
     ($descr:literal) => {
-        $crate::panic(concat!("internal error: ", $descr))
+        panic!(concat!("internal error: ", $descr))
     };
 }
 

time/src/lib.rs

@@ -149,11 +149,3 @@ pub use crate::weekday::Weekday;
 
 /// An alias for [`std::result::Result`] with a generic error from the time crate.
 pub type Result<T> = core::result::Result<T, Error>;
-
-/// This is a separate function to reduce the code size of explicit panics.
-#[inline(never)]
-#[cold]
-#[track_caller]
-const fn panic(message: &str) -> ! {
-    panic!("{}", message)
-}