Remove a suggestion that is redundant
This commit is contained in:
parent
c763f833d1
commit
6018e21d8a
@ -118,7 +118,6 @@ pub fn check_match(
|
|||||||
prior_arm_ty,
|
prior_arm_ty,
|
||||||
prior_arm_span,
|
prior_arm_span,
|
||||||
scrut_span: scrut.span,
|
scrut_span: scrut.span,
|
||||||
scrut_hir_id: scrut.hir_id,
|
|
||||||
source: match_src,
|
source: match_src,
|
||||||
prior_non_diverging_arms: prior_non_diverging_arms.clone(),
|
prior_non_diverging_arms: prior_non_diverging_arms.clone(),
|
||||||
opt_suggest_box_span,
|
opt_suggest_box_span,
|
||||||
|
@ -780,7 +780,6 @@ fn note_error_origin(
|
|||||||
ref prior_non_diverging_arms,
|
ref prior_non_diverging_arms,
|
||||||
opt_suggest_box_span,
|
opt_suggest_box_span,
|
||||||
scrut_span,
|
scrut_span,
|
||||||
scrut_hir_id,
|
|
||||||
..
|
..
|
||||||
}) => match source {
|
}) => match source {
|
||||||
hir::MatchSource::TryDesugar(scrut_hir_id) => {
|
hir::MatchSource::TryDesugar(scrut_hir_id) => {
|
||||||
@ -848,18 +847,6 @@ fn note_error_origin(
|
|||||||
) {
|
) {
|
||||||
err.subdiagnostic(subdiag);
|
err.subdiagnostic(subdiag);
|
||||||
}
|
}
|
||||||
if let hir::Node::Expr(m) = self.tcx.parent_hir_node(scrut_hir_id)
|
|
||||||
&& let hir::Node::Stmt(stmt) = self.tcx.parent_hir_node(m.hir_id)
|
|
||||||
&& let hir::StmtKind::Expr(_) = stmt.kind
|
|
||||||
{
|
|
||||||
err.span_suggestion_verbose(
|
|
||||||
stmt.span.shrink_to_hi(),
|
|
||||||
"consider using a semicolon here, but this will discard any values \
|
|
||||||
in the match arms",
|
|
||||||
";",
|
|
||||||
Applicability::MaybeIncorrect,
|
|
||||||
);
|
|
||||||
}
|
|
||||||
if let Some(ret_sp) = opt_suggest_box_span {
|
if let Some(ret_sp) = opt_suggest_box_span {
|
||||||
// Get return type span and point to it.
|
// Get return type span and point to it.
|
||||||
self.suggest_boxing_for_return_impl_trait(
|
self.suggest_boxing_for_return_impl_trait(
|
||||||
|
@ -569,7 +569,6 @@ pub struct MatchExpressionArmCause<'tcx> {
|
|||||||
pub prior_arm_ty: Ty<'tcx>,
|
pub prior_arm_ty: Ty<'tcx>,
|
||||||
pub prior_arm_span: Span,
|
pub prior_arm_span: Span,
|
||||||
pub scrut_span: Span,
|
pub scrut_span: Span,
|
||||||
pub scrut_hir_id: hir::HirId,
|
|
||||||
pub source: hir::MatchSource,
|
pub source: hir::MatchSource,
|
||||||
pub prior_non_diverging_arms: Vec<Span>,
|
pub prior_non_diverging_arms: Vec<Span>,
|
||||||
pub opt_suggest_box_span: Option<Span>,
|
pub opt_suggest_box_span: Option<Span>,
|
||||||
|
@ -15,10 +15,6 @@ LL | | }
|
|||||||
|
|
|
|
||||||
= note: expected enum `Option<u16>`
|
= note: expected enum `Option<u16>`
|
||||||
found unit type `()`
|
found unit type `()`
|
||||||
help: consider using a semicolon here, but this will discard any values in the match arms
|
|
||||||
|
|
|
||||||
LL | };
|
|
||||||
| +
|
|
||||||
|
|
||||||
error: aborting due to 1 previous error
|
error: aborting due to 1 previous error
|
||||||
|
|
||||||
|
@ -4,22 +4,15 @@ error[E0308]: `match` arms have incompatible types
|
|||||||
LL | / match num {
|
LL | / match num {
|
||||||
LL | | 1 => {
|
LL | | 1 => {
|
||||||
LL | | cx.answer_str("hi");
|
LL | | cx.answer_str("hi");
|
||||||
| | -------------------- this is found to be of type `()`
|
| | --------------------
|
||||||
|
| | | |
|
||||||
|
| | | help: consider removing this semicolon
|
||||||
|
| | this is found to be of type `()`
|
||||||
LL | | }
|
LL | | }
|
||||||
LL | | _ => cx.answer_str("hi"),
|
LL | | _ => cx.answer_str("hi"),
|
||||||
| | ^^^^^^^^^^^^^^^^^^^ expected `()`, found future
|
| | ^^^^^^^^^^^^^^^^^^^ expected `()`, found future
|
||||||
LL | | }
|
LL | | }
|
||||||
| |_____- `match` arms have incompatible types
|
| |_____- `match` arms have incompatible types
|
||||||
|
|
|
||||||
help: consider removing this semicolon
|
|
||||||
|
|
|
||||||
LL - cx.answer_str("hi");
|
|
||||||
LL + cx.answer_str("hi")
|
|
||||||
|
|
|
||||||
help: consider using a semicolon here, but this will discard any values in the match arms
|
|
||||||
|
|
|
||||||
LL | };
|
|
||||||
| +
|
|
||||||
|
|
||||||
error: aborting due to 1 previous error
|
error: aborting due to 1 previous error
|
||||||
|
|
||||||
|
@ -11,10 +11,6 @@ LL | | }
|
|||||||
|
|
|
|
||||||
= note: expected reference `&S`
|
= note: expected reference `&S`
|
||||||
found reference `&R`
|
found reference `&R`
|
||||||
help: consider using a semicolon here, but this will discard any values in the match arms
|
|
||||||
|
|
|
||||||
LL | };
|
|
||||||
| +
|
|
||||||
|
|
||||||
error[E0038]: the trait `Trait` cannot be made into an object
|
error[E0038]: the trait `Trait` cannot be made into an object
|
||||||
--> $DIR/wf-unsafe-trait-obj-match.rs:26:21
|
--> $DIR/wf-unsafe-trait-obj-match.rs:26:21
|
||||||
|
Loading…
Reference in New Issue
Block a user