address comments

rust-lang · Mar 6, 2021 · 7d3a6f1 · 7d3a6f1
1 parent d9d6921
commit 7d3a6f1
Show file tree

Hide file tree

Showing 7 changed files with 13 additions and 14 deletions.
diff --git a/compiler/rustc_typeck/src/collect/type_of.rs b/compiler/rustc_typeck/src/collect/type_of.rs
@@ -659,18 +659,20 @@ fn infer_placeholder_type(
                 format!("{}: {}", item_ident, ty),
                 Applicability::MachineApplicable,
             )
-            .emit_unless(matches!(ty.kind(), ty::Error(_)));
+            .emit_unless(ty.references_error());
         }
         None => {
             let mut diag = bad_placeholder_type(tcx, vec![span]);
-            if !matches!(ty.kind(), ty::Error(_)) {
+
+            if !ty.references_error() {
                 diag.span_suggestion(
                     span,
                     "replace `_` with the correct type",
                     ty.to_string(),
                     Applicability::MaybeIncorrect,
                 );
             }
+
             diag.emit();
         }
     }

diff --git a/src/test/ui/79040.rs b/src/test/ui/79040.rs
diff --git a/src/test/ui/parser/item-free-const-no-body-semantic-fail.rs b/src/test/ui/parser/item-free-const-no-body-semantic-fail.rs
@@ -4,4 +4,3 @@ fn main() {}
 
 const A: u8; //~ ERROR free constant item without body
 const B; //~ ERROR free constant item without body
-//~^ ERROR missing type for `const` item
diff --git a/src/test/ui/parser/item-free-static-no-body-semantic-fail.rs b/src/test/ui/parser/item-free-static-no-body-semantic-fail.rs
@@ -4,8 +4,6 @@ fn main() {}
 
 static A: u8; //~ ERROR free static item without body
 static B; //~ ERROR free static item without body
-//~^ ERROR missing type for `static` item
 
 static mut C: u8; //~ ERROR free static item without body
 static mut D; //~ ERROR free static item without body
-//~^ ERROR missing type for `static mut` item
diff --git a/src/test/ui/parser/item-free-static-no-body-semantic-fail.stderr b/src/test/ui/parser/item-free-static-no-body-semantic-fail.stderr
@@ -15,15 +15,15 @@ LL | static B;
    |         help: provide a definition for the static: `= <expr>;`
 
 error: free static item without body
-  --> $DIR/item-free-static-no-body-semantic-fail.rs:9:1
+  --> $DIR/item-free-static-no-body-semantic-fail.rs:8:1
    |
 LL | static mut C: u8;
    | ^^^^^^^^^^^^^^^^-
    |                 |
    |                 help: provide a definition for the static: `= <expr>;`
 
 error: free static item without body
-  --> $DIR/item-free-static-no-body-semantic-fail.rs:10:1
+  --> $DIR/item-free-static-no-body-semantic-fail.rs:9:1
    |
 LL | static mut D;
    | ^^^^^^^^^^^^-

diff --git a/src/test/ui/typeck/issue-79040.rs b/src/test/ui/typeck/issue-79040.rs
@@ -0,0 +1,5 @@
+fn main() {
+    const FOO = "hello" + 1; //~ ERROR cannot add `{integer}` to `&str`
+    //~^ ERROR cannot add `{integer}` to `&str`
+    println!("{}", FOO);
+}
diff --git a/src/test/ui/79040.stderr → src/test/ui/typeck/issue-79040.stderr b/src/test/ui/79040.stderr → src/test/ui/typeck/issue-79040.stderr
@@ -1,13 +1,13 @@
 error[E0369]: cannot add `{integer}` to `&str`
-  --> $DIR/79040.rs:2:25
+  --> $DIR/issue-79040.rs:2:25
    |
 LL |     const FOO = "hello" + 1;
    |                 ------- ^ - {integer}
    |                 |
    |                 &str
 
 error[E0369]: cannot add `{integer}` to `&str`
-  --> $DIR/79040.rs:2:25
+  --> $DIR/issue-79040.rs:2:25
    |
 LL |     const FOO = "hello" + 1;
    |                 ------- ^ - {integer}