Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

rust: Inline format args #1030

Open
wants to merge 1 commit into
base: rust
Choose a base branch
from
Open

rust: Inline format args #1030

wants to merge 1 commit into from

Commits on Aug 15, 2023

  1. rust: Inline format args 

    A minor Rust cleanup to inline `format!("{}", foo)` and similar macro calls into `format!("{foo}")`.

While mostly cosmetic, this change makes it a bit easier to read format argument strings, shortens code, and makes it easier to see cases with double-referencing values causing minor performance (apparently `&foo` in a format! causes about 6% perf cost due to Rust compiler being unable to inline it due to dynamic invocation.
    @nyurik
    nyurik committed Aug 15, 2023
    Configuration menu
    Copy the full SHA
    c23e18f View commit details
    Browse the repository at this point in the history