Skip to content

Commit 622997e

Browse files
committed
refactor: simplify pattern
1 parent 1b7266a commit 622997e

File tree

1 file changed

+11
-8
lines changed

1 file changed

+11
-8
lines changed

parser/src/nullable.rs

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -99,14 +99,17 @@ fn get_used_table_name<'a>(table_name: &str, from: &'a ast::FromClause) -> Optio
9999

100100
fn test_expr(column_name: &str, table_name: &str, expr: &ast::Expr) -> Option<NullableResult> {
101101
match expr {
102-
ast::Expr::Binary(left, ast::Operator::Equals, right)
103-
| ast::Expr::Binary(left, ast::Operator::NotEquals, right)
104-
| ast::Expr::Binary(left, ast::Operator::Greater, right)
105-
| ast::Expr::Binary(left, ast::Operator::GreaterEquals, right)
106-
| ast::Expr::Binary(left, ast::Operator::Less, right)
107-
| ast::Expr::Binary(left, ast::Operator::LessEquals, right)
108-
if expr_matches_name(column_name, table_name, left)
109-
|| expr_matches_name(column_name, table_name, right) =>
102+
ast::Expr::Binary(
103+
left,
104+
ast::Operator::Equals
105+
| ast::Operator::NotEquals
106+
| ast::Operator::Greater
107+
| ast::Operator::GreaterEquals
108+
| ast::Operator::Less
109+
| ast::Operator::LessEquals,
110+
right,
111+
) if expr_matches_name(column_name, table_name, left)
112+
|| expr_matches_name(column_name, table_name, right) =>
110113
{
111114
return Some(NullableResult::NotNull);
112115
}

0 commit comments

Comments
 (0)