Ignore println newline in foreign format hints

Author: Dnreikronos

compiler/rustc_builtin_macros/src/format.rs

@@ -607,6 +607,8 @@ fn make_format_args(
         // If there's a lot of unused arguments,
         // let's check if this format arguments looks like another syntax (printf / shell).
         let detect_foreign_fmt = unused.len() > args.explicit_args().len() / 2;
+        let foreign_fmt_str =
+            if append_newline { fmt_str.strip_suffix('\n').unwrap_or(fmt_str) } else { fmt_str };
         report_missing_placeholders(
             ecx,
             unused,
@@ -616,7 +618,7 @@ fn make_format_args(
             &invalid_refs,
             detect_foreign_fmt,
             str_style,
-            fmt_str,
+            foreign_fmt_str,
             uncooked_fmt_str.1.as_str(),
             fmt_span,
         );

tests/ui/macros/format-foreign-dollar-without-spec.stderr

@@ -2,15 +2,14 @@ error: argument never used
   --> $DIR/format-foreign-dollar-without-spec.rs:3:25
    |
 LL |     println!("%65536$", 1);
-   |                         ^ argument never used
+   |              ---------  ^ argument never used
+   |              |
+   |              formatting specifier missing
    |
-note: format specifiers use curly braces, and the conversion specifier `
-      ` is unknown or unsupported
-  --> $DIR/format-foreign-dollar-without-spec.rs:3:15
+help: format specifiers use curly braces, consider adding a format specifier
    |
-LL |     println!("%65536$", 1);
-   |               ^^^^^^^^
-   = note: printf formatting is not supported; see the documentation for `std::fmt`
+LL |     println!("%65536${}", 1);
+   |                       ++
 
 error: aborting due to 1 previous error
 

tests/ui/macros/issue-158216.rs

@@ -0,0 +1,6 @@
+//@ check-fail
+
+pub fn f(x: f64) {
+    println!("{x:>8.2}%", "foo");
+    //~^ ERROR argument never used
+}

tests/ui/macros/issue-158216.stderr

@@ -0,0 +1,14 @@
+error: argument never used
+  --> $DIR/issue-158216.rs:4:27
+   |
+LL |     println!("{x:>8.2}%", "foo");
+   |              -----------  ^^^^^ argument never used
+   |              |
+   |              formatting specifier missing
+   |
+help: format specifiers use curly braces, consider adding a format specifier
+   |
+LL |     println!("{x:>8.2}%{}", "foo");
+   |                       ++
+
+error: aborting due to 1 previous error

tests/ui/macros/issue-92267.stderr

@@ -2,15 +2,13 @@ error: argument never used
   --> $DIR/issue-92267.rs:3:34
    |
 LL | pub fn main() { println!("🦀%%%", 0) }
-   |                                   ^ argument never used
+   |                          --------  ^ argument never used
+   |                          |
+   |                          formatting specifier missing
    |
-note: format specifiers use curly braces, and the conversion specifier `
-      ` is unknown or unsupported
-  --> $DIR/issue-92267.rs:3:30
+help: format specifiers use curly braces, consider adding a format specifier
    |
-LL | pub fn main() { println!("🦀%%%", 0) }
-   |                               ^^
-   = note: printf formatting is not supported; see the documentation for `std::fmt`
+LL | pub fn main() { println!("🦀%%%{}", 0) }
+   |                                 ++
 
 error: aborting due to 1 previous error
-