Applicability adjustment per additional comments
This commit is contained in:
parent
58729346be
commit
52fb7d461e
3 changed files with 3 additions and 3 deletions
|
|
@ -102,7 +102,7 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for LargeEnumVariant {
|
|||
"consider boxing the large fields to reduce the total size of the \
|
||||
enum",
|
||||
format!("Box<{}>", snip),
|
||||
Applicability::MachineApplicable,
|
||||
Applicability::Unspecified,
|
||||
);
|
||||
return;
|
||||
}
|
||||
|
|
|
|||
|
|
@ -125,7 +125,7 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for LetIfSeq {
|
|||
span,
|
||||
"it is more idiomatic to write",
|
||||
sug,
|
||||
Applicability::MaybeIncorrect,
|
||||
Applicability::HasPlaceholders,
|
||||
);
|
||||
if !mutability.is_empty() {
|
||||
db.note("you might not need `mut` at all");
|
||||
|
|
|
|||
|
|
@ -344,7 +344,7 @@ fn check_match_bool(cx: &LateContext<'_, '_>, ex: &Expr, arms: &[Arm], expr: &Ex
|
|||
expr.span,
|
||||
"consider using an if/else expression",
|
||||
sugg,
|
||||
Applicability::MaybeIncorrect, // not sure
|
||||
Applicability::HasPlaceholders,
|
||||
);
|
||||
}
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue