mathieu/granite-rust
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Actually fix the lint applied

Browse source
This commit is contained in:
scott-linder 2017-02-02 08:36:40 -05:00
parent a4c4da1c4b
commit 80cb48ca1a
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
clippy_lints/src/types.rs
View file

@ -142,7 +142,7 @@ fn check_ty(cx: &LateContext, ast_ty: &hir::Ty) {
match_def_path(cx.tcx, did, &paths::VEC), match_def_path(cx.tcx, did, &paths::VEC),
], { ], {
span_help_and_lint(cx, span_help_and_lint(cx,
BORROWED_BOX, BOX_VEC,
ast_ty.span, ast_ty.span,
"you seem to be trying to use `Box<Vec<T>>`. Consider using just `Vec<T>`", "you seem to be trying to use `Box<Vec<T>>`. Consider using just `Vec<T>`",
"`Vec<T>` is already on the heap, `Box<Vec<T>>` makes an extra allocation."); "`Vec<T>` is already on the heap, `Box<Vec<T>>` makes an extra allocation.");
@ -184,7 +184,7 @@ fn check_ty(cx: &LateContext, ast_ty: &hir::Ty) {
if let Some(def_id) = opt_def_id(def) { if let Some(def_id) = opt_def_id(def) {
if Some(def_id) == cx.tcx.lang_items.owned_box() { if Some(def_id) == cx.tcx.lang_items.owned_box() {
span_help_and_lint(cx, span_help_and_lint(cx,
BOX_VEC, BORROWED_BOX,
ast_ty.span, ast_ty.span,
"you seem to be trying to use `&Box<T>`. Consider using just `&T`", "you seem to be trying to use `&Box<T>`. Consider using just `&T`",
"replace `&Box<T>` with simply `&T`"); "replace `&Box<T>` with simply `&T`");