Use triple (in)equalities when type compatibility is ensured

bperel · bperel · commit ef7968e5a80d · 2018-11-26T20:09:00.000+01:00
diff --git a/src/Components/AlterOperation.php b/src/Components/AlterOperation.php
@@ -236,7 +236,7 @@ public static function parse(Parser $parser, TokensList $list, array $options =
                     // We have reached the end of ALTER operation and suddenly found
                     // a start to new statement, but have not find a delimiter between them
 
-                    if (!($token->value == 'SET' && $list->tokens[$list->idx - 1]->value == 'CHARACTER')) {
+                    if (!($token->value === 'SET' && $list->tokens[$list->idx - 1]->value === 'CHARACTER')) {
                         $parser->error(
                             'A new statement was found, but no delimiter between it and the previous one.',
                             $token
diff --git a/src/Components/Array2d.php b/src/Components/Array2d.php
@@ -82,7 +82,7 @@ public static function parse(Parser $parser, TokensList $list, array $options =
                     $arrCount = count($arr->values);
                     if ($count === -1) {
                         $count = $arrCount;
-                    } elseif ($arrCount != $count) {
+                    } elseif ($arrCount !== $count) {
                         $parser->error(
                             sprintf(
                                 Translator::gettext('%1$d values were expected, but found %2$d.'),
diff --git a/src/Components/Condition.php b/src/Components/Condition.php
@@ -179,7 +179,7 @@ public static function parse(Parser $parser, TokensList $list, array $options =
                 if ($token->value === '(') {
                     ++$brackets;
                 } elseif ($token->value === ')') {
-                    if ($brackets == 0) {
+                    if ($brackets === 0) {
                         break;
                     }
                     --$brackets;
diff --git a/src/Components/Expression.php b/src/Components/Expression.php
@@ -299,7 +299,7 @@ public static function parse(Parser $parser, TokensList $list, array $options =
                         $ret->function = $prev[1]->value;
                     }
                 } elseif ($token->value === ')') {
-                    if ($brackets == 0) {
+                    if ($brackets === 0) {
                         // Not our bracket
                         break;
                     } else {
diff --git a/src/Components/IntoKeyword.php b/src/Components/IntoKeyword.php
@@ -207,10 +207,10 @@ public static function parse(Parser $parser, TokensList $list, array $options =
                 $ret->dest = $token->value;
 
                 $state = 3;
-            } elseif ($state == 3) {
+            } elseif ($state === 3) {
                 $ret->parseFileOptions($parser, $list, $token->value);
                 $state = 4;
-            } elseif ($state == 4) {
+            } elseif ($state === 4) {
                 if ($token->type === Token::TYPE_KEYWORD && $token->keyword !== 'LINES') {
                     break;
                 }
diff --git a/src/Components/OptionsArray.php b/src/Components/OptionsArray.php
@@ -247,10 +247,10 @@ public static function parse(Parser $parser, TokensList $list, array $options =
          */
         if ($state === 1
             && $lastOption
-            && ($lastOption[1] == 'expr'
-            || $lastOption[1] == 'var'
-            || $lastOption[1] == 'var='
-            || $lastOption[1] == 'expr=')
+            && ($lastOption[1] === 'expr'
+            || $lastOption[1] === 'var'
+            || $lastOption[1] === 'var='
+            || $lastOption[1] === 'expr=')
         ) {
             $parser->error(
                 sprintf(
diff --git a/src/Components/SetOperation.php b/src/Components/SetOperation.php
@@ -94,7 +94,7 @@ public static function parse(Parser $parser, TokensList $list, array $options =
             // No keyword is expected.
             if (($token->type === Token::TYPE_KEYWORD)
                 && ($token->flags & Token::FLAG_KEYWORD_RESERVED)
-                && ($state == 0)
+                && ($state === 0)
             ) {
                 break;
             }
@@ -113,7 +113,7 @@ public static function parse(Parser $parser, TokensList $list, array $options =
                         'breakOnAlias' => true,
                     )
                 );
-                if ($tmp == null) {
+                if (is_null($tmp)) {
                     $parser->error('Missing expression.', $token);
                     break;
                 }
diff --git a/src/Context.php b/src/Context.php
@@ -313,7 +313,7 @@ public static function isWhitespace($str)
     public static function isComment($str, $end = false)
     {
         $len = strlen($str);
-        if ($len == 0) {
+        if ($len === 0) {
             return null;
         }
         if ($str[0] === '#') {
@@ -382,7 +382,7 @@ public static function isNumber($str)
      */
     public static function isSymbol($str)
     {
-        if (strlen($str) == 0) {
+        if (strlen($str) === 0) {
             return null;
         }
         if ($str[0] === '@') {
@@ -408,7 +408,7 @@ public static function isSymbol($str)
      */
     public static function isString($str)
     {
-        if (strlen($str) == 0) {
+        if (strlen($str) === 0) {
             return null;
         }
         if ($str[0] === '\'') {
diff --git a/src/Statement.php b/src/Statement.php
@@ -451,9 +451,7 @@ public function validateClauseOrder($parser, $list)
     {
         $clauses = array_flip(array_keys($this->getClauses()));
 
-        if (empty($clauses)
-            || count($clauses) == 0
-        ) {
+        if (empty($clauses) || count($clauses) === 0) {
             return true;
         }
 
@@ -489,7 +487,7 @@ public function validateClauseOrder($parser, $list)
             );
 
             // Handle ordering of Multiple Joins in a query
-            if ($clauseStartIdx != -1) {
+            if ($clauseStartIdx !== -1) {
                 if ($minJoin === 0 && stripos($clauseType, 'JOIN')) {
                     // First JOIN clause is detected
                     $minJoin = $maxJoin = $clauseStartIdx;
@@ -501,7 +499,7 @@ public function validateClauseOrder($parser, $list)
                 }
             }
 
-            if ($clauseStartIdx != -1 && $clauseStartIdx < $minIdx) {
+            if ($clauseStartIdx !== -1 && $clauseStartIdx < $minIdx) {
                 if ($minJoin === 0 || $error === 1) {
                     $token = $list->tokens[$clauseStartIdx];
                     $parser->error(
@@ -512,7 +510,7 @@ public function validateClauseOrder($parser, $list)
                     return false;
                 }
                 $minIdx = $clauseStartIdx;
-            } elseif ($clauseStartIdx != -1) {
+            } elseif ($clauseStartIdx !== -1) {
                 $minIdx = $clauseStartIdx;
             }
 
diff --git a/src/Statements/CreateStatement.php b/src/Statements/CreateStatement.php
@@ -388,7 +388,7 @@ public function parse(Parser $parser, TokensList $list)
          */
         $token = $list->tokens[$list->idx];
         $nextidx = $list->idx + 1;
-        while ($nextidx < $list->count && $list->tokens[$nextidx]->type == Token::TYPE_WHITESPACE) {
+        while ($nextidx < $list->count && $list->tokens[$nextidx]->type === Token::TYPE_WHITESPACE) {
             ++$nextidx;
         }
 
@@ -399,18 +399,18 @@ public function parse(Parser $parser, TokensList $list)
                 static::$DB_OPTIONS
             );
         } elseif ($this->options->has('TABLE')) {
-            if (($token->type == Token::TYPE_KEYWORD)
-             && ($token->keyword == 'SELECT')) {
+            if (($token->type === Token::TYPE_KEYWORD)
+             && ($token->keyword === 'SELECT')) {
                 /* CREATE TABLE ... SELECT */
                 $this->select = new SelectStatement($parser, $list);
-            } elseif (($token->type == Token::TYPE_KEYWORD) && ($token->keyword == 'AS')
-                && ($list->tokens[$nextidx]->type == Token::TYPE_KEYWORD)
-                && ($list->tokens[$nextidx]->value == 'SELECT')) {
+            } elseif (($token->type === Token::TYPE_KEYWORD) && ($token->keyword === 'AS')
+                && ($list->tokens[$nextidx]->type === Token::TYPE_KEYWORD)
+                && ($list->tokens[$nextidx]->value === 'SELECT')) {
                 /* CREATE TABLE ... AS SELECT */
                 $list->idx = $nextidx;
                 $this->select = new SelectStatement($parser, $list);
-            } elseif ($token->type == Token::TYPE_KEYWORD
-                && $token->keyword == 'LIKE') {
+            } elseif ($token->type === Token::TYPE_KEYWORD
+                && $token->keyword === 'LIKE') {
             /* CREATE TABLE `new_tbl` LIKE 'orig_tbl' */
                 $list->idx = $nextidx;
                 $this->like = Expression::parse(
@@ -422,7 +422,7 @@ public function parse(Parser $parser, TokensList $list)
                     )
                 );
                 // The 'LIKE' keyword was found, but no table_name was found next to it
-                if ($this->like == null) {
+                if (is_null($this->like)) {
                     $parser->error(
                         'A table name was expected.',
                         $list->tokens[$list->idx]
diff --git a/src/Statements/DeleteStatement.php b/src/Statements/DeleteStatement.php
@@ -142,25 +142,25 @@ public function build()
     {
         $ret = 'DELETE ' . OptionsArray::build($this->options);
 
-        if ($this->columns != null && count($this->columns) > 0) {
+        if (!is_null($this->columns) && count($this->columns) > 0) {
             $ret .= ' ' . ExpressionArray::build($this->columns);
         }
-        if ($this->from != null && count($this->from) > 0) {
+        if (!is_null($this->from) && count($this->from) > 0) {
             $ret .= ' FROM ' . ExpressionArray::build($this->from);
         }
-        if ($this->join != null && count($this->join) > 0) {
+        if (!is_null($this->join) && count($this->join) > 0) {
             $ret .= ' ' . JoinKeyword::build($this->join);
         }
-        if ($this->using != null && count($this->using) > 0) {
+        if (!is_null($this->using) && count($this->using) > 0) {
             $ret .= ' USING ' . ExpressionArray::build($this->using);
         }
-        if ($this->where != null && count($this->where) > 0) {
+        if (!is_null($this->where) && count($this->where) > 0) {
             $ret .= ' WHERE ' . Condition::build($this->where);
         }
-        if ($this->order != null && count($this->order) > 0) {
+        if (!is_null($this->order) && count($this->order) > 0) {
             $ret .= ' ORDER BY ' . ExpressionArray::build($this->order);
         }
-        if ($this->limit != null && strlen($this->limit) > 0) {
+        if (!is_null($this->limit) && strlen($this->limit) > 0) {
             $ret .= ' LIMIT ' . Limit::build($this->limit);
         }
 
diff --git a/src/Statements/InsertStatement.php b/src/Statements/InsertStatement.php
@@ -113,15 +113,15 @@ public function build()
         $ret = 'INSERT ' . $this->options
             . ' INTO ' . $this->into;
 
-        if ($this->values != null && count($this->values) > 0) {
+        if (!is_null($this->values) && count($this->values) > 0) {
             $ret .= ' VALUES ' . Array2d::build($this->values);
-        } elseif ($this->set != null && count($this->set) > 0) {
+        } elseif (!is_null($this->set) && count($this->set) > 0) {
             $ret .= ' SET ' . SetOperation::build($this->set);
-        } elseif ($this->select != null && strlen($this->select) > 0) {
+        } elseif (!is_null($this->select) && strlen($this->select) > 0) {
             $ret .= ' ' . $this->select->build();
         }
 
-        if ($this->onDuplicateSet != null && count($this->onDuplicateSet) > 0) {
+        if (!is_null($this->onDuplicateSet) && count($this->onDuplicateSet) > 0) {
             $ret .= ' ON DUPLICATE KEY UPDATE ' . SetOperation::build($this->onDuplicateSet);
         }
 
@@ -229,7 +229,7 @@ public function parse(Parser $parser, TokensList $list)
                     );
                     break;
                 }
-            } elseif ($state == 2) {
+            } elseif ($state === 2) {
                 $lastCount = $miniState;
 
                 if ($miniState === 1 && $token->keyword === 'ON') {
diff --git a/src/Statements/ReplaceStatement.php b/src/Statements/ReplaceStatement.php
@@ -89,11 +89,11 @@ public function build()
     {
         $ret = 'REPLACE ' . $this->options . ' INTO ' . $this->into;
 
-        if ($this->values != null && count($this->values) > 0) {
+        if (!is_null($this->values) && count($this->values) > 0) {
             $ret .= ' VALUES ' . Array2d::build($this->values);
-        } elseif ($this->set != null && count($this->set) > 0) {
+        } elseif (!is_null($this->set) && count($this->set) > 0) {
             $ret .= ' SET ' . SetOperation::build($this->set);
-        } elseif ($this->select != null && strlen($this->select) > 0) {
+        } elseif (!is_null($this->select) && strlen($this->select) > 0) {
             $ret .= ' ' . $this->select->build();
         }
 
diff --git a/src/Utils/BufferedQuery.php b/src/Utils/BufferedQuery.php
@@ -191,7 +191,7 @@ public function extract($end = false)
              * treated differently, because of the preceding backslash, it will
              * be ignored.
              */
-            if ((($this->status & static::STATUS_COMMENT) == 0) && ($this->query[$i] === '\\')) {
+            if ((($this->status & static::STATUS_COMMENT) === 0) && ($this->query[$i] === '\\')) {
                 $this->current .= $this->query[$i] . $this->query[++$i];
                 continue;
             }
@@ -319,7 +319,7 @@ public function extract($end = false)
                 }
 
                 // Checking if the delimiter definition ended.
-                if (($delimiter != '')
+                if (($delimiter !== '')
                     && ((($i < $len) && Context::isWhitespace($this->query[$i]))
                     || (($i === $len) && $end))
                 ) {
diff --git a/src/Utils/CLI.php b/src/Utils/CLI.php
@@ -120,7 +120,7 @@ public function runLint()
             $lexer = new Lexer($params['q'], false);
             $parser = new Parser($lexer->list);
             $errors = Error::get(array($lexer, $parser));
-            if (count($errors) == 0) {
+            if (count($errors) === 0) {
                 return 0;
             }
             $output = Error::format($errors);
diff --git a/src/Utils/Formatter.php b/src/Utils/Formatter.php
@@ -554,7 +554,7 @@ public function toString($token)
                 if ($this->options['type'] === 'html') {
                     return '<span ' . $format['html'] . '>' . htmlspecialchars($text, ENT_NOQUOTES) . '</span>';
                 } elseif ($this->options['type'] === 'cli') {
-                    if ($prev != $format['cli']) {
+                    if ($prev !== $format['cli']) {
                         $prev = $format['cli'];
 
                         return $format['cli'] . $this->escapeConsole($text);
@@ -568,7 +568,7 @@ public function toString($token)
         }
 
         if ($this->options['type'] === 'cli') {
-            if ($prev != "\x1b[39m") {
+            if ($prev !== "\x1b[39m") {
                 $prev = "\x1b[39m";
 
                 return "\x1b[39m" . $this->escapeConsole($text);
@@ -633,7 +633,7 @@ public static function getGroupLength($list)
                     ++$count;
                 } elseif ($list->tokens[$idx]->value === ')') {
                     --$count;
-                    if ($count == 0) {
+                    if ($count === 0) {
                         break;
                     }
                 }
diff --git a/src/Utils/Query.php b/src/Utils/Query.php
@@ -633,7 +633,7 @@ public static function getClause($statement, $list, $clause, $type = 0, $skipFir
                     && ($clauses[$token->keyword] >= $currIdx)
                 ) {
                     $currIdx = $clauses[$token->keyword];
-                    if ($skipFirst && ($currIdx == $clauseIdx)) {
+                    if ($skipFirst && ($currIdx === $clauseIdx)) {
                         // This token is skipped (not added to the old
                         // clause) because it will be replaced.
                         continue;
@@ -843,7 +843,7 @@ public static function getClauseStartOffset($statement, $list, $clause)
                 }
             }
 
-            if ($brackets == 0) {
+            if ($brackets === 0) {
                 if (($token->type === Token::TYPE_KEYWORD)
                     && isset($clauses[$token->keyword])
                     && ($clause === $token->keyword)
diff --git a/tools/ContextGenerator.php b/tools/ContextGenerator.php
@@ -131,7 +131,7 @@ public static function readWords(array $files)
 
         $types = array();
 
-        for ($i = 0, $count = count($words); $i != $count; ++$i) {
+        for ($i = 0, $count = count($words); $i !== $count; ++$i) {
             $type = 1;
             $value = trim($words[$i]);
 
@@ -198,17 +198,17 @@ public static function printWords($words, $spaces = 8, $line = 80)
                 $i = 0;
 
                 foreach ($wordsByLen as $word) {
-                    if ($i == 0) {
+                    if ($i === 0) {
                         $ret .= str_repeat(' ', $spaces);
                     }
                     $ret .= sprintf('\'%s\' => %s, ', $word, $type);
-                    if (++$i == $count) {
+                    if (++$i === $count) {
                         $ret .= "\n";
                         $i = 0;
                     }
                 }
 
-                if ($i != 0) {
+                if ($i !== 0) {
                     $ret .= "\n";
                 }
             }
@@ -272,7 +272,7 @@ public static function formatName($name)
 
         /* Parse version to array */
         $ver_str = $parts[2];
-        if (strlen($ver_str) % 2 == 1) {
+        if (strlen($ver_str) % 2 === 1) {
             $ver_str = '0' . $ver_str;
         }
         $version = array_map('intval', str_split($ver_str, 2));

Original file line number	Diff line number	Diff line change
`@@ -179,7 +179,7 @@ public static function parse(Parser $parser, TokensList $list, array $options =`
`179`	`179`	`if ($token->value === '(') {`
`180`	`180`	`++$brackets;`
`181`	`181`	`} elseif ($token->value === ')') {`
`182`		`- if ($brackets == 0) {`
	`182`	`+ if ($brackets === 0) {`
`183`	`183`	`break;`
`184`	`184`	`}`
`185`	`185`	`--$brackets;`
Original file line number	Diff line number	Diff line change
`@@ -299,7 +299,7 @@ public static function parse(Parser $parser, TokensList $list, array $options =`
`299`	`299`	`$ret->function = $prev[1]->value;`
`300`	`300`	`}`
`301`	`301`	`} elseif ($token->value === ')') {`
`302`		`- if ($brackets == 0) {`
	`302`	`+ if ($brackets === 0) {`
`303`	`303`	`// Not our bracket`
`304`	`304`	`break;`
`305`	`305`	`} else {`
Original file line number	Diff line number	Diff line change
`@@ -94,7 +94,7 @@ public static function parse(Parser $parser, TokensList $list, array $options =`
`94`	`94`	`// No keyword is expected.`
`95`	`95`	`if (($token->type === Token::TYPE_KEYWORD)`
`96`	`96`	`&& ($token->flags & Token::FLAG_KEYWORD_RESERVED)`
`97`		`- && ($state == 0)`
	`97`	`+ && ($state === 0)`
`98`	`98`	`) {`
`99`	`99`	`break;`
`100`	`100`	`}`
`@@ -113,7 +113,7 @@ public static function parse(Parser $parser, TokensList $list, array $options =`
`113`	`113`	`'breakOnAlias' => true,`
`114`	`114`	`)`
`115`	`115`	`);`
`116`		`- if ($tmp == null) {`
	`116`	`+ if (is_null($tmp)) {`
`117`	`117`	`$parser->error('Missing expression.', $token);`
`118`	`118`	`break;`
`119`	`119`	`}`
Original file line number	Diff line number	Diff line change
`@@ -313,7 +313,7 @@ public static function isWhitespace($str)`
`313`	`313`	`public static function isComment($str, $end = false)`
`314`	`314`	`{`
`315`	`315`	`$len = strlen($str);`
`316`		`- if ($len == 0) {`
	`316`	`+ if ($len === 0) {`
`317`	`317`	`return null;`
`318`	`318`	`}`
`319`	`319`	`if ($str[0] === '#') {`
`@@ -382,7 +382,7 @@ public static function isNumber($str)`
`382`	`382`	`*/`
`383`	`383`	`public static function isSymbol($str)`
`384`	`384`	`{`
`385`		`- if (strlen($str) == 0) {`
	`385`	`+ if (strlen($str) === 0) {`
`386`	`386`	`return null;`
`387`	`387`	`}`
`388`	`388`	`if ($str[0] === '@') {`
`@@ -408,7 +408,7 @@ public static function isSymbol($str)`
`408`	`408`	`*/`
`409`	`409`	`public static function isString($str)`
`410`	`410`	`{`
`411`		`- if (strlen($str) == 0) {`
	`411`	`+ if (strlen($str) === 0) {`
`412`	`412`	`return null;`
`413`	`413`	`}`
`414`	`414`	`if ($str[0] === '\'') {`
Original file line number	Diff line number	Diff line change
`@@ -451,9 +451,7 @@ public function validateClauseOrder($parser, $list)`
`451`	`451`	`{`
`452`	`452`	`$clauses = array_flip(array_keys($this->getClauses()));`
`453`	`453`
`454`		`- if (empty($clauses)`
`455`		`- \|\| count($clauses) == 0`
`456`		`- ) {`
	`454`	`+ if (empty($clauses) \|\| count($clauses) === 0) {`
`457`	`455`	`return true;`
`458`	`456`	`}`
`459`	`457`
`@@ -489,7 +487,7 @@ public function validateClauseOrder($parser, $list)`
`489`	`487`	`);`
`490`	`488`
`491`	`489`	`// Handle ordering of Multiple Joins in a query`
`492`		`- if ($clauseStartIdx != -1) {`
	`490`	`+ if ($clauseStartIdx !== -1) {`
`493`	`491`	`if ($minJoin === 0 && stripos($clauseType, 'JOIN')) {`
`494`	`492`	`// First JOIN clause is detected`
`495`	`493`	`$minJoin = $maxJoin = $clauseStartIdx;`
`@@ -501,7 +499,7 @@ public function validateClauseOrder($parser, $list)`
`501`	`499`	`}`
`502`	`500`	`}`
`503`	`501`
`504`		`- if ($clauseStartIdx != -1 && $clauseStartIdx < $minIdx) {`
	`502`	`+ if ($clauseStartIdx !== -1 && $clauseStartIdx < $minIdx) {`
`505`	`503`	`if ($minJoin === 0 \|\| $error === 1) {`
`506`	`504`	`$token = $list->tokens[$clauseStartIdx];`
`507`	`505`	`$parser->error(`
`@@ -512,7 +510,7 @@ public function validateClauseOrder($parser, $list)`
`512`	`510`	`return false;`
`513`	`511`	`}`
`514`	`512`	`$minIdx = $clauseStartIdx;`
`515`		`- } elseif ($clauseStartIdx != -1) {`
	`513`	`+ } elseif ($clauseStartIdx !== -1) {`
`516`	`514`	`$minIdx = $clauseStartIdx;`
`517`	`515`	`}`
`518`	`516`