Pretty print break and continue without redundant space

[dtolnay]

Repro:

macro_rules! m { ($e:expr) => { stringify!($e) }; } fn main() { println!("{:?}", m!(loop { break; })); println!("{:?}", m!(loop { break 'a; })); println!("{:?}", m!(loop { break false; })); }

Before:

• "loop { break ; }"
• "loop { break 'a ; }"
• "loop { break false ; }"

After:

• "loop { break; }"
• "loop { break 'a; }"
• "loop { break false; }"

Notice that return and yield already follow the same approach as this PR of printing the space before each additional piece following the keyword, rather than after each thing.

rust/compiler/rustc_ast_pretty/src/pprust/state.rs

Lines 2148 to 2154 in 772d51f

	ast::ExprKind::Ret(ref result) => {
	self.s.word("return");
	if let Some(ref expr) = *result {
	self.s.word(" ");
	self.print_expr_maybe_paren(expr, parser::PREC_JUMP);
	}
	}

rust/compiler/rustc_ast_pretty/src/pprust/state.rs

Lines 2221 to 2228 in 772d51f

	ast::ExprKind::Yield(ref e) => {
	self.s.word("yield");
	if let Some(ref expr) = *e {
	self.s.space();
	self.print_expr_maybe_paren(expr, parser::PREC_JUMP);
	}
	}

[rust-highfive]

r? @nagisa

(rust-highfive has picked a reviewer for you, use r? to override)

[nagisa]

@bors r+

[bors]

📌 Commit abb8893 has been approved by nagisa

[bors]

☔ The latest upstream changes (presumably #91665) made this pull request unmergeable. Please resolve the merge conflicts.

[dtolnay]

• Rebased to resolve conflict with Remove unneeded access to pretty printer's s field in favor of deref #91571

[dtolnay]

@bors r=nagisa

[bors]

📌 Commit f0f7b8d has been approved by nagisa