Skip to content

Commit

Permalink
Rollup merge of #93357 - nnethercote:clarify-usage-of-qualified-ty, r…
Browse files Browse the repository at this point in the history
…=lcnr

Clarify the `usage-of-qualified-ty` error message.

I found this message confusing when I encountered it. This commit makes
it clearer that you have to import the unqualified type yourself.

r? `@lcnr`
  • Loading branch information
matthiaskrgr authored Jan 27, 2022
2 parents b232c46 + 696d503 commit 5d79874
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion compiler/rustc_lint/src/internal.rs
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ impl<'tcx> LateLintPass<'tcx> for TyTyKind {
lint.build(&format!("usage of qualified `ty::{}`", t))
.span_suggestion(
path.span,
"try using it unqualified",
"try importing it and using it unqualified",
t,
// The import probably needs to be changed
Applicability::MaybeIncorrect,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ error: usage of qualified `ty::Ty<'_>`
--> $DIR/qualified_ty_ty_ctxt.rs:25:11
|
LL | ty_q: ty::Ty<'_>,
| ^^^^^^^^^^ help: try using it unqualified: `Ty<'_>`
| ^^^^^^^^^^ help: try importing it and using it unqualified: `Ty<'_>`
|
note: the lint level is defined here
--> $DIR/qualified_ty_ty_ctxt.rs:4:9
Expand All @@ -14,7 +14,7 @@ error: usage of qualified `ty::TyCtxt<'_>`
--> $DIR/qualified_ty_ty_ctxt.rs:27:16
|
LL | ty_ctxt_q: ty::TyCtxt<'_>,
| ^^^^^^^^^^^^^^ help: try using it unqualified: `TyCtxt<'_>`
| ^^^^^^^^^^^^^^ help: try importing it and using it unqualified: `TyCtxt<'_>`

error: aborting due to 2 previous errors

0 comments on commit 5d79874

Please sign in to comment.