Skip to content

Commit

Permalink
Remove duplicate switch cases for code clarity (#303)
Browse files Browse the repository at this point in the history
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
  • Loading branch information
TysonAndre authored and greenlion committed Jun 1, 2019
1 parent a4ad4ea commit 28ff145
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
1 change: 0 additions & 1 deletion src/PHPSQLParser/processors/ExpressionListProcessor.php
Expand Up @@ -274,7 +274,6 @@ public function process($tokens) {
case 'AND':
case '&&':
case 'BETWEEN':
case 'AND':
case 'BINARY':
case '&':
case '~':
Expand Down
1 change: 0 additions & 1 deletion src/PHPSQLParser/processors/SQLProcessor.php
Expand Up @@ -494,7 +494,6 @@ public function process($tokens) {

case '':
case ',':
case ';':
break;

default:
Expand Down

0 comments on commit 28ff145

Please sign in to comment.