Skip to content

Commit

Permalink
Rollup merge of rust-lang#122963 - RalfJung:core-panicking, r=m-ou-se
Browse files Browse the repository at this point in the history
core/panicking: fix outdated comment

Looks like this function got renamed/changed at some point and the comment did not get updated.

r? ``@m-ou-se``
  • Loading branch information
workingjubilee authored Mar 24, 2024
2 parents 4ea53c0 + e74b01e commit 2dcdbfd
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions library/core/src/panicking.rs
Original file line number Diff line number Diff line change
Expand Up @@ -132,11 +132,11 @@ pub const fn panic_nounwind_fmt(fmt: fmt::Arguments<'_>, force_no_backtrace: boo
#[rustc_const_unstable(feature = "panic_internals", issue = "none")]
#[lang = "panic"] // needed by codegen for panic on overflow and other `Assert` MIR terminators
pub const fn panic(expr: &'static str) -> ! {
// Use Arguments::new_v1 instead of format_args!("{expr}") to potentially
// Use Arguments::new_const instead of format_args!("{expr}") to potentially
// reduce size overhead. The format_args! macro uses str's Display trait to
// write expr, which calls Formatter::pad, which must accommodate string
// truncation and padding (even though none is used here). Using
// Arguments::new_v1 may allow the compiler to omit Formatter::pad from the
// Arguments::new_const may allow the compiler to omit Formatter::pad from the
// output binary, saving up to a few kilobytes.
panic_fmt(fmt::Arguments::new_const(&[expr]));
}
Expand Down

0 comments on commit 2dcdbfd

Please sign in to comment.