]> git.lizzy.rs Git - rust.git/commitdiff
review comments: do not always point at init expr
authorEsteban Küber <esteban@kuber.com.ar>
Tue, 3 Jan 2023 07:34:26 +0000 (23:34 -0800)
committerEsteban Küber <esteban@kuber.com.ar>
Thu, 5 Jan 2023 16:51:16 +0000 (16:51 +0000)
20 files changed:
compiler/rustc_hir_typeck/src/demand.rs
src/test/ui/argument-suggestions/two-mismatch-notes.stderr
src/test/ui/async-await/dont-suggest-missing-await.stderr
src/test/ui/async-await/suggest-missing-await-closure.stderr
src/test/ui/async-await/suggest-missing-await.stderr
src/test/ui/closures/closure-return-type-mismatch.stderr
src/test/ui/coercion/coerce-to-bang.stderr
src/test/ui/generic-associated-types/collections-project-default.stderr
src/test/ui/issues/issue-15783.stderr
src/test/ui/let-else/let-else-ref-bindings.stderr
src/test/ui/methods/method-deref-to-same-trait-object-with-separate-params.stderr
src/test/ui/mismatched_types/abridged.stderr
src/test/ui/parser/struct-literal-variant-in-if.stderr
src/test/ui/rfc-2497-if-let-chains/ensure-that-let-else-does-not-interact-with-let-chains.stderr
src/test/ui/structs/struct-base-wrong-type.stderr
src/test/ui/suggestions/call-boxed.stderr
src/test/ui/suggestions/issue-86100-tuple-paren-comma.stderr
src/test/ui/traits/issue-52893.stderr
src/test/ui/tuple/wrong_argument_ice-3.stderr
src/test/ui/type/type-mismatch-same-crate-name.stderr

index 5ba0d41ab9d1194d70f5f961e8ee4f0b65638326..c50d03b944ffd8a1536015114e137b1b63a599da 100644 (file)
@@ -355,6 +355,9 @@ fn visit_expr(&mut self, ex: &'v hir::Expr<'v>) {
                 }
             }
         }
+        if span_labels.len() < 2 {
+            return false;
+        }
         for (sp, label) in span_labels {
             err.span_label(sp, &label);
         }
index 3ccd399863d556e2692a3b24497c5cb0196ca77b..7873cf964cbbc978b278bf3cf58b0a5320152d66 100644 (file)
@@ -1,8 +1,6 @@
 error[E0308]: arguments to this function are incorrect
   --> $DIR/two-mismatch-notes.rs:10:5
    |
-LL |     let w = Wrapper::<isize>(1isize);
-   |             ------------------------ here the type of `w` is inferred to be `Wrapper<isize>`
 LL |     foo(f, w);
    |     ^^^
    |
index 6e232dd006426fcd72381080a7feb9a4025b383e..627bf05bba2d9c77110d0f7b00af849089c8b5d1 100644 (file)
@@ -1,8 +1,6 @@
 error[E0308]: mismatched types
   --> $DIR/dont-suggest-missing-await.rs:14:18
    |
-LL |         let x = make_u32();
-   |                 ---------- here the type of `x` is inferred to be `impl Future<Output = u32>`
 LL |         take_u32(x)
    |         -------- ^ expected `u32`, found opaque type
    |         |
index 9f51832365b615d7170fe451da928c66e5960d89..a5958baffbaf72719eb8ba87c6f3f17163c2ea4b 100644 (file)
@@ -1,8 +1,6 @@
 error[E0308]: mismatched types
   --> $DIR/suggest-missing-await-closure.rs:16:18
    |
-LL |         let x = make_u32();
-   |                 ---------- here the type of `x` is inferred to be `impl Future<Output = u32>`
 LL |         take_u32(x)
    |         -------- ^ expected `u32`, found opaque type
    |         |
index ce4c8edaf74d6d2d1e1633efdfa5c003f0f0ce2b..1196601ace09087eb0433a807758f949eb2657d7 100644 (file)
@@ -1,8 +1,6 @@
 error[E0308]: mismatched types
   --> $DIR/suggest-missing-await.rs:12:14
    |
-LL |     let x = make_u32();
-   |             ---------- here the type of `x` is inferred to be `impl Future<Output = u32>`
 LL |     take_u32(x)
    |     -------- ^ expected `u32`, found opaque type
    |     |
index d33cf383cbcb4ca38d5e550dd6ffccfff6faa956..3a89d30a05d2042b7106730d21aabcf5b20b2306 100644 (file)
@@ -1,8 +1,6 @@
 error[E0308]: mismatched types
   --> $DIR/closure-return-type-mismatch.rs:7:9
    |
-LL |         let a = true;
-   |                 ---- here the type of `a` is inferred to be `bool`
 LL |         a
    |         ^ expected `&str`, found `bool`
    |
index d2fd0f788384eda78315d144a2d2e469e53ba1d0..1207dc7e7a2ff4415da00c5e039b3e4b16418d39 100644 (file)
@@ -33,9 +33,6 @@ LL | fn foo(x: usize, y: !, z: usize) { }
 error[E0308]: mismatched types
   --> $DIR/coerce-to-bang.rs:26:12
    |
-LL |     let b = 22;
-   |             -- here the type of `b` is inferred to be `{integer}`
-LL |     let c = 44;
 LL |     foo(a, b, c); // ... and hence a reference to `a` is expected to diverge.
    |     ---    ^ expected `!`, found integer
    |     |
@@ -52,9 +49,6 @@ LL | fn foo(x: usize, y: !, z: usize) { }
 error[E0308]: mismatched types
   --> $DIR/coerce-to-bang.rs:36:12
    |
-LL |     let b = 22;
-   |             -- here the type of `b` is inferred to be `{integer}`
-LL |     let c = 44;
 LL |     foo(a, b, c);
    |     ---    ^ expected `!`, found integer
    |     |
@@ -71,9 +65,6 @@ LL | fn foo(x: usize, y: !, z: usize) { }
 error[E0308]: mismatched types
   --> $DIR/coerce-to-bang.rs:45:12
    |
-LL |     let b = 22;
-   |             -- here the type of `b` is inferred to be `{integer}`
-LL |     let c = 44;
 LL |     foo(a, b, c);
    |     ---    ^ expected `!`, found integer
    |     |
index c11a5acc35244fbac426d91025372eb37c9eda2a..5701017dc3471adad820c7ec5268b5725cc2f13a 100644 (file)
@@ -4,9 +4,6 @@ error[E0308]: mismatched types
 LL | fn floatify_sibling<C>(ints: &C) -> <C as Collection<i32>>::Sibling<f32>
    |                                     ------------------------------------ expected `<C as Collection<i32>>::Sibling<f32>` because of return type
 ...
-LL |     let mut res = <C::Family as CollectionFamily>::Member::<f32>::empty();
-   |                   ------------------------------------------------------- here the type of `res` is inferred to be `<<C as Collection<i32>>::Family as CollectionFamily>::Member<f32>`
-...
 LL |     res
    |     ^^^ expected Collection::Sibling, found CollectionFamily::Member
    |
index e7fecfebdc8f3f3bc303e5c1b1daa94611688822..660dfe9ed3d51bc7617b139fd900ba1b773f3137 100644 (file)
@@ -1,8 +1,6 @@
 error[E0308]: mismatched types
   --> $DIR/issue-15783.rs:8:19
    |
-LL |     let x = Some(&[name]);
-   |             ------------- here the type of `x` is inferred to be `Option<&[&str; 1]>`
 LL |     let msg = foo(x);
    |               --- ^ expected slice `[&str]`, found array `[&str; 1]`
    |               |
index 4c58b2db575d02620f9a3596053772e7ebe35382..56b9e073330a6237817715f5c4827f70d6f30fad 100644 (file)
@@ -19,9 +19,6 @@ LL |     let Some(ref a): Option<&[u8]> = &some else { return };
 error[E0308]: mismatched types
   --> $DIR/let-else-ref-bindings.rs:24:34
    |
-LL |     let some = Some(bytes);
-   |                ----------- here the type of `some` is inferred to be `Option<Vec<u8>>`
-...
 LL |     let Some(a): Option<&[u8]> = some else { return };
    |                  -------------   ^^^^ expected `&[u8]`, found struct `Vec`
    |                  |
@@ -62,9 +59,6 @@ LL |     let Some(ref mut a): Option<&mut [u8]> = &mut some else { return };
 error[E0308]: mismatched types
   --> $DIR/let-else-ref-bindings.rs:52:38
    |
-LL |     let mut some = Some(bytes);
-   |                    ----------- here the type of `some` is inferred to be `Option<Vec<u8>>`
-...
 LL |     let Some(a): Option<&mut [u8]> = some else { return };
    |                  -----------------   ^^^^ expected `&mut [u8]`, found struct `Vec`
    |                  |
index 1d836f20012720699916dabf37111d8e8c4ad960..82addab94792ac06c4ce8ba5c9563451d42ff514 100644 (file)
@@ -10,9 +10,6 @@ LL | #![feature(unsized_locals, unsized_fn_params)]
 error[E0308]: mismatched types
   --> $DIR/method-deref-to-same-trait-object-with-separate-params.rs:87:24
    |
-LL |     let z = x.foo();
-   |             ------- here the type of `z` is inferred to be `u32`
-...
 LL |     let _seetype: () = z;
    |                   --   ^ expected `()`, found `u32`
    |                   |
@@ -21,9 +18,6 @@ LL |     let _seetype: () = z;
 error[E0308]: mismatched types
   --> $DIR/method-deref-to-same-trait-object-with-separate-params.rs:104:24
    |
-LL |     let z = x.foo();
-   |             ------- here the type of `z` is inferred to be `u64`
-...
 LL |     let _seetype: () = z;
    |                   --   ^ expected `()`, found `u64`
    |                   |
@@ -66,9 +60,6 @@ LL |     let z = FinalFoo::foo(x);
 error[E0308]: mismatched types
   --> $DIR/method-deref-to-same-trait-object-with-separate-params.rs:139:24
    |
-LL |     let z = x.foo();
-   |             ------- here the type of `z` is inferred to be `u8`
-...
 LL |     let _seetype: () = z;
    |                   --   ^ expected `()`, found `u8`
    |                   |
@@ -77,9 +68,6 @@ LL |     let _seetype: () = z;
 error[E0308]: mismatched types
   --> $DIR/method-deref-to-same-trait-object-with-separate-params.rs:157:24
    |
-LL |     let z = x.foo();
-   |             ------- here the type of `z` is inferred to be `u32`
-...
 LL |     let _seetype: () = z;
    |                   --   ^ expected `()`, found `u32`
    |                   |
@@ -88,9 +76,6 @@ LL |     let _seetype: () = z;
 error[E0308]: mismatched types
   --> $DIR/method-deref-to-same-trait-object-with-separate-params.rs:174:24
    |
-LL |     let z = x.foo();
-   |             ------- here the type of `z` is inferred to be `u32`
-...
 LL |     let _seetype: () = z;
    |                   --   ^ expected `()`, found `u32`
    |                   |
index 9b5f0134f127470d7b2592ce1dd02521e9e4344a..ff1a836c9aec034b7e2cbd5e929b5a190a05b89a 100644 (file)
@@ -53,19 +53,11 @@ LL |     Ok(Foo { bar: 1 })
 error[E0308]: mismatched types
   --> $DIR/abridged.rs:39:5
    |
-LL |   fn d() -> X<X<String, String>, String> {
-   |             ---------------------------- expected `X<X<String, String>, String>` because of return type
-LL |       let x = X {
-   |  _____________-
-LL | |         x: X {
-LL | |             x: "".to_string(),
-LL | |             y: 2,
-LL | |         },
-LL | |         y: 3,
-LL | |     };
-   | |_____- here the type of `x` is inferred to be `X<X<String, {integer}>, {integer}>`
-LL |       x
-   |       ^ expected struct `String`, found integer
+LL | fn d() -> X<X<String, String>, String> {
+   |           ---------------------------- expected `X<X<String, String>, String>` because of return type
+...
+LL |     x
+   |     ^ expected struct `String`, found integer
    |
    = note: expected struct `X<X<_, String>, String>`
               found struct `X<X<_, {integer}>, {integer}>`
@@ -73,19 +65,11 @@ LL |       x
 error[E0308]: mismatched types
   --> $DIR/abridged.rs:50:5
    |
-LL |   fn e() -> X<X<String, String>, String> {
-   |             ---------------------------- expected `X<X<String, String>, String>` because of return type
-LL |       let x = X {
-   |  _____________-
-LL | |         x: X {
-LL | |             x: "".to_string(),
-LL | |             y: 2,
-LL | |         },
-LL | |         y: "".to_string(),
-LL | |     };
-   | |_____- here the type of `x` is inferred to be `X<X<String, {integer}>, String>`
-LL |       x
-   |       ^ expected struct `String`, found integer
+LL | fn e() -> X<X<String, String>, String> {
+   |           ---------------------------- expected `X<X<String, String>, String>` because of return type
+...
+LL |     x
+   |     ^ expected struct `String`, found integer
    |
    = note: expected struct `X<X<_, String>, _>`
               found struct `X<X<_, {integer}>, _>`
index 97cdd130d0bec0862a3b9b76e1dfa5b71a30ac3e..9f0c0074d674ceac43093dc128f3cf1266f9a550 100644 (file)
@@ -51,8 +51,6 @@ LL |     if x == E::V { field } {}
 error[E0308]: mismatched types
   --> $DIR/struct-literal-variant-in-if.rs:10:20
    |
-LL |     let field = true;
-   |                 ---- here the type of `field` is inferred to be `bool`
 LL |     if x == E::V { field } {}
    |     ---------------^^^^^--
    |     |              |
index 8c30f015c5d84fc18f5645dd103803d0a595bb54..498a112fa9bb3f7434e6031827821cf9ba38bb69 100644 (file)
@@ -100,9 +100,6 @@ LL |     let Some(n) = opt && let another = n else {
 error[E0308]: mismatched types
   --> $DIR/ensure-that-let-else-does-not-interact-with-let-chains.rs:9:19
    |
-LL |     let opt = Some(1i32);
-   |               ---------- here the type of `opt` is inferred to be `Option<i32>`
-...
 LL |     let Some(n) = opt && n == 1 else {
    |                   ^^^ expected `bool`, found enum `Option`
    |
@@ -123,9 +120,6 @@ LL |     let Some(n) = opt && n == 1 else {
 error[E0308]: mismatched types
   --> $DIR/ensure-that-let-else-does-not-interact-with-let-chains.rs:15:19
    |
-LL |     let opt = Some(1i32);
-   |               ---------- here the type of `opt` is inferred to be `Option<i32>`
-...
 LL |     let Some(n) = opt && let another = n else {
    |                   ^^^ expected `bool`, found enum `Option`
    |
index 30feb9cdd70c117f86c28ee92c521bca26881d4a..b039ce2cc9209fc81f5e967dd762c032c5ec57c3 100644 (file)
@@ -13,8 +13,6 @@ LL | static foo_i: Foo = Foo { a: 2, ..4 };
 error[E0308]: mismatched types
   --> $DIR/struct-base-wrong-type.rs:12:27
    |
-LL |     let b = Bar { x: 5 };
-   |             ------------ here the type of `b` is inferred to be `Bar`
 LL |     let f = Foo { a: 2, ..b };
    |                           ^ expected struct `Foo`, found struct `Bar`
 
index 1609c2a3094e11a0efa94673dc31f528be06ee6c..9b619ac9a3f502fcc4983655dec8070c4362bd77 100644 (file)
@@ -4,10 +4,7 @@ error[E0308]: mismatched types
 LL |     let mut x = 1i32;
    |                 ---- expected due to this value
 LL |     let y = Box::new(|| 1);
-   |             --------------
-   |             |        |
-   |             |        the found closure
-   |             here the type of `y` is inferred to be `Box<[closure@call-boxed.rs:3:22]>`
+   |                      -- the found closure
 LL |     x = y;
    |         ^ expected `i32`, found struct `Box`
    |
index 388d8d8d895e323e4851f687e773e4b74da081a2..8c9a41a202767813306bf508a328f785ad74e45c 100644 (file)
@@ -49,8 +49,6 @@ LL |     let _s = S { _s: ("abc".to_string(),) };
 error[E0308]: mismatched types
   --> $DIR/issue-86100-tuple-paren-comma.rs:23:22
    |
-LL |     let t = (1, 2);
-   |             ------ here the type of `t` is inferred to be `({integer}, {integer})`
 LL |     let _x: (i32,) = (t);
    |             ------   ^^^ expected a tuple with 1 element, found one with 2 elements
    |             |
index f7b5b7fca06e87db9eca21ab204d26219a9f6bde..7924d3db06f36efb7ce39b5e0740200fc312099e 100644 (file)
@@ -4,8 +4,6 @@ error[E0308]: mismatched types
 LL | impl<F, Name, P> AddClass<Name, F> for Class<P>
    |      - this type parameter
 ...
-LL |         let output = builder.to_ref();
-   |                      ---------------- here the type of `output` is inferred to be `Class<P>`
 LL |         builder.push(output);
    |                 ---- ^^^^^^ expected type parameter `F`, found struct `Class`
    |                 |
index 1ad1d692121022738626f7e4c07c2811e8b21d92..0a503e1fe58c1a503c2e81fb136d0e70368dd24d 100644 (file)
@@ -1,9 +1,6 @@
 error[E0061]: this method takes 1 argument but 2 arguments were supplied
   --> $DIR/wrong_argument_ice-3.rs:9:16
    |
-LL |     let new_group = vec![String::new()];
-   |                     ------------------- here the type of `new_group` is inferred to be `Vec<String>`
-...
 LL |         groups.push(new_group, vec![process]);
    |                ^^^^            ------------- argument of type `Vec<&Process>` unexpected
    |
index 0d754459f4a5053a65141840ccddf7e7efa91a54..fcafd315ebf5450e5f0ea68e6281a6d52969bf46 100644 (file)
@@ -1,9 +1,6 @@
 error[E0308]: mismatched types
   --> $DIR/type-mismatch-same-crate-name.rs:16:20
    |
-LL |     let foo2 = {extern crate crate_a2 as a; a::Foo};
-   |                ------------------------------------ here the type of `foo2` is inferred to be `Foo`
-...
 LL |         a::try_foo(foo2);
    |         ---------- ^^^^ expected struct `main::a::Foo`, found a different struct `main::a::Foo`
    |         |
@@ -30,9 +27,6 @@ LL | pub fn try_foo(x: Foo){}
 error[E0308]: mismatched types
   --> $DIR/type-mismatch-same-crate-name.rs:20:20
    |
-LL |     let bar2 = {extern crate crate_a2 as a; a::bar()};
-   |                -------------------------------------- here the type of `bar2` is inferred to be `Box<dyn Bar>`
-...
 LL |         a::try_bar(bar2);
    |         ---------- ^^^^ expected trait `main::a::Bar`, found a different trait `main::a::Bar`
    |         |