Rename BinOpKind::lazy
as BinOpKind::is_lazy
.
To match `BinOpKind::is_comparison` and `hir::BinOpKind::is_lazy`.
This commit is contained in:
parent
0efd2a9d8f
commit
705b484922
3 changed files with 5 additions and 4 deletions
|
@ -881,7 +881,8 @@ impl BinOpKind {
|
||||||
Gt => ">",
|
Gt => ">",
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
pub fn lazy(&self) -> bool {
|
|
||||||
|
pub fn is_lazy(&self) -> bool {
|
||||||
matches!(self, BinOpKind::And | BinOpKind::Or)
|
matches!(self, BinOpKind::And | BinOpKind::Or)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -656,7 +656,7 @@ trait UnusedDelimLint {
|
||||||
) -> bool {
|
) -> bool {
|
||||||
if followed_by_else {
|
if followed_by_else {
|
||||||
match inner.kind {
|
match inner.kind {
|
||||||
ast::ExprKind::Binary(op, ..) if op.node.lazy() => return true,
|
ast::ExprKind::Binary(op, ..) if op.node.is_lazy() => return true,
|
||||||
_ if classify::expr_trailing_brace(inner).is_some() => return true,
|
_ if classify::expr_trailing_brace(inner).is_some() => return true,
|
||||||
_ => {}
|
_ => {}
|
||||||
}
|
}
|
||||||
|
@ -1016,7 +1016,7 @@ impl UnusedDelimLint for UnusedParens {
|
||||||
rustc_span::source_map::Spanned { node, .. },
|
rustc_span::source_map::Spanned { node, .. },
|
||||||
_,
|
_,
|
||||||
_,
|
_,
|
||||||
) if node.lazy()))
|
) if node.is_lazy()))
|
||||||
{
|
{
|
||||||
self.emit_unused_delims_expr(cx, value, ctx, left_pos, right_pos, is_kw)
|
self.emit_unused_delims_expr(cx, value, ctx, left_pos, right_pos, is_kw)
|
||||||
}
|
}
|
||||||
|
|
|
@ -384,7 +384,7 @@ impl<'a> Parser<'a> {
|
||||||
|
|
||||||
fn check_let_else_init_bool_expr(&self, init: &ast::Expr) {
|
fn check_let_else_init_bool_expr(&self, init: &ast::Expr) {
|
||||||
if let ast::ExprKind::Binary(op, ..) = init.kind {
|
if let ast::ExprKind::Binary(op, ..) = init.kind {
|
||||||
if op.node.lazy() {
|
if op.node.is_lazy() {
|
||||||
self.sess.emit_err(errors::InvalidExpressionInLetElse {
|
self.sess.emit_err(errors::InvalidExpressionInLetElse {
|
||||||
span: init.span,
|
span: init.span,
|
||||||
operator: op.node.as_str(),
|
operator: op.node.as_str(),
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue