Skip to content

Commit

Permalink
Rollup merge of #91818 - camelid:unused-result-type, r=jackh726
Browse files Browse the repository at this point in the history
Show the unused type for `unused_results` lint

I think it's helpful to know what type was unused when looking at these
warnings. The type will likely determine whether the result *should* be
used, or whether it should just be ignored.

Including the type also matches the behavior of the `must_use` lint:
unused `SomeType` that must be used.
  • Loading branch information
matthiaskrgr committed Dec 18, 2021
2 parents 7a626cf + f53e489 commit 24b75e7
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
4 changes: 3 additions & 1 deletion compiler/rustc_lint/src/unused.rs
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,9 @@ impl<'tcx> LateLintPass<'tcx> for UnusedResults {
}

if !(type_permits_lack_of_use || fn_warned || op_warned) {
cx.struct_span_lint(UNUSED_RESULTS, s.span, |lint| lint.build("unused result").emit());
cx.struct_span_lint(UNUSED_RESULTS, s.span, |lint| {
lint.build(&format!("unused result of type `{}`", ty)).emit()
});
}

// Returns whether an error has been emitted (and thus another does not need to be later).
Expand Down
2 changes: 1 addition & 1 deletion src/test/ui/lint/unused/unused-result.rs
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ fn test2() {
}

fn main() {
foo::<isize>(); //~ ERROR: unused result
foo::<isize>(); //~ ERROR: unused result of type `isize`
foo::<MustUse>(); //~ ERROR: unused `MustUse` that must be used
foo::<MustUseMsg>(); //~ ERROR: unused `MustUseMsg` that must be used
//~^ NOTE: some message
Expand Down
2 changes: 1 addition & 1 deletion src/test/ui/lint/unused/unused-result.stderr
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ LL | foo::<MustUseMsg>();
|
= note: some message

error: unused result
error: unused result of type `isize`
--> $DIR/unused-result.rs:34:5
|
LL | foo::<isize>();
Expand Down

0 comments on commit 24b75e7

Please sign in to comment.