From 28ff1451a093bba0c42dec8fed8199d8b06a6fd0 Mon Sep 17 00:00:00 2001 From: Tyson Andre Date: Sat, 1 Jun 2019 00:02:44 -0400 Subject: [PATCH] Remove duplicate switch cases for code clarity (#303) If there are multiple switch cases, only the earliest one is used. On line 444 of SQLProcessor, `case ';'` was a `continue 2;` instead. Detected via static analysis --- src/PHPSQLParser/processors/ExpressionListProcessor.php | 1 - src/PHPSQLParser/processors/SQLProcessor.php | 1 - 2 files changed, 2 deletions(-) diff --git a/src/PHPSQLParser/processors/ExpressionListProcessor.php b/src/PHPSQLParser/processors/ExpressionListProcessor.php index 8b7ee6b1..b0353f76 100644 --- a/src/PHPSQLParser/processors/ExpressionListProcessor.php +++ b/src/PHPSQLParser/processors/ExpressionListProcessor.php @@ -274,7 +274,6 @@ public function process($tokens) { case 'AND': case '&&': case 'BETWEEN': - case 'AND': case 'BINARY': case '&': case '~': diff --git a/src/PHPSQLParser/processors/SQLProcessor.php b/src/PHPSQLParser/processors/SQLProcessor.php index a3348456..4c8ccfc3 100644 --- a/src/PHPSQLParser/processors/SQLProcessor.php +++ b/src/PHPSQLParser/processors/SQLProcessor.php @@ -494,7 +494,6 @@ public function process($tokens) { case '': case ',': - case ';': break; default: