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.

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

Remove println! in rewrite_static() #6214

Closed
wants to merge 1 commit into from

Conversation

Creative0708
Copy link

There's an extraneous println!() in rewrite_static in items.rs:

rustfmt/src/items.rs

Lines 1992 to 2000 in e494418

fn rewrite_static(
context: &RewriteContext<'_>,
static_parts: &StaticParts<'_>,
offset: Indent,
) -> Option<String> {
println!("rewriting static");
let colon = colon_spaces(context.config);
let mut prefix = format!(
"{}{}{}{} {}{}{}",

This causes rustfmt to add "rewriting static" to the top of every file that triggers rewrite_static() when formatting:

image

This PR removes the println!(). Nothing fancy, I assume this was a mistake.

@Creative0708
Copy link
Author

Oops, sorry. Didn't see #6210 for some reason.

@Creative0708 Creative0708 deleted the remove-println branch June 24, 2024 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants