Enable the fixer enforcing fully-qualified calls for compiler-optimized functions

stof · stof · commit c2dfbf9a7182 · 2018-07-24T12:05:38.000+02:00
diff --git a/Node/AbstractNode.php b/Node/AbstractNode.php
@@ -34,7 +34,7 @@ abstract class AbstractNode implements NodeInterface
     public function getNodeName()
     {
         if (null === $this->nodeName) {
-            $this->nodeName = preg_replace('~.*\\\\([^\\\\]+)Node$~', '$1', get_called_class());
+            $this->nodeName = preg_replace('~.*\\\\([^\\\\]+)Node$~', '$1', \get_called_class());
         }
 
         return $this->nodeName;
diff --git a/Parser/Handler/HashHandler.php b/Parser/Handler/HashHandler.php
@@ -51,7 +51,7 @@ public function handle(Reader $reader, TokenStream $stream)
 
         $value = $this->escaping->escapeUnicode($match[1]);
         $stream->push(new Token(Token::TYPE_HASH, $value, $reader->getPosition()));
-        $reader->moveForward(strlen($match[0]));
+        $reader->moveForward(\strlen($match[0]));
 
         return true;
     }
diff --git a/Parser/Handler/IdentifierHandler.php b/Parser/Handler/IdentifierHandler.php
@@ -51,7 +51,7 @@ public function handle(Reader $reader, TokenStream $stream)
 
         $value = $this->escaping->escapeUnicode($match[0]);
         $stream->push(new Token(Token::TYPE_IDENTIFIER, $value, $reader->getPosition()));
-        $reader->moveForward(strlen($match[0]));
+        $reader->moveForward(\strlen($match[0]));
 
         return true;
     }
diff --git a/Parser/Handler/NumberHandler.php b/Parser/Handler/NumberHandler.php
@@ -47,7 +47,7 @@ public function handle(Reader $reader, TokenStream $stream)
         }
 
         $stream->push(new Token(Token::TYPE_NUMBER, $match[0], $reader->getPosition()));
-        $reader->moveForward(strlen($match[0]));
+        $reader->moveForward(\strlen($match[0]));
 
         return true;
     }
diff --git a/Parser/Handler/StringHandler.php b/Parser/Handler/StringHandler.php
@@ -47,7 +47,7 @@ public function handle(Reader $reader, TokenStream $stream)
     {
         $quote = $reader->getSubstring(1);
 
-        if (!in_array($quote, array("'", '"'))) {
+        if (!\in_array($quote, array("'", '"'))) {
             return false;
         }
 
@@ -59,18 +59,18 @@ public function handle(Reader $reader, TokenStream $stream)
         }
 
         // check unclosed strings
-        if (strlen($match[0]) === $reader->getRemainingLength()) {
+        if (\strlen($match[0]) === $reader->getRemainingLength()) {
             throw SyntaxErrorException::unclosedString($reader->getPosition() - 1);
         }
 
         // check quotes pairs validity
-        if ($quote !== $reader->getSubstring(1, strlen($match[0]))) {
+        if ($quote !== $reader->getSubstring(1, \strlen($match[0]))) {
             throw SyntaxErrorException::unclosedString($reader->getPosition() - 1);
         }
 
         $string = $this->escaping->escapeUnicodeAndNewLine($match[0]);
         $stream->push(new Token(Token::TYPE_STRING, $string, $reader->getPosition()));
-        $reader->moveForward(strlen($match[0]) + 1);
+        $reader->moveForward(\strlen($match[0]) + 1);
 
         return true;
     }
diff --git a/Parser/Handler/WhitespaceHandler.php b/Parser/Handler/WhitespaceHandler.php
@@ -39,7 +39,7 @@ public function handle(Reader $reader, TokenStream $stream)
         }
 
         $stream->push(new Token(Token::TYPE_WHITESPACE, $match[0], $reader->getPosition()));
-        $reader->moveForward(strlen($match[0]));
+        $reader->moveForward(\strlen($match[0]));
 
         return true;
     }
diff --git a/Parser/Parser.php b/Parser/Parser.php
@@ -169,7 +169,7 @@ private function parseSimpleSelector(TokenStream $stream, $insideNegation = fals
     {
         $stream->skipWhitespace();
 
-        $selectorStart = count($stream->getUsed());
+        $selectorStart = \count($stream->getUsed());
         $result = $this->parseElementNode($stream);
         $pseudoElement = null;
 
@@ -206,7 +206,7 @@ private function parseSimpleSelector(TokenStream $stream, $insideNegation = fals
                 }
 
                 $identifier = $stream->getNextIdentifier();
-                if (in_array(strtolower($identifier), array('first-line', 'first-letter', 'before', 'after'))) {
+                if (\in_array(strtolower($identifier), array('first-line', 'first-letter', 'before', 'after'))) {
                     // Special case: CSS 2.1 pseudo-elements can have a single ':'.
                     // Any new pseudo-element must have two.
                     $pseudoElement = $identifier;
@@ -272,7 +272,7 @@ private function parseSimpleSelector(TokenStream $stream, $insideNegation = fals
             }
         }
 
-        if (count($stream->getUsed()) === $selectorStart) {
+        if (\count($stream->getUsed()) === $selectorStart) {
             throw SyntaxErrorException::unexpectedToken('selector', $stream->getPeek());
         }
 
diff --git a/Parser/Reader.php b/Parser/Reader.php
@@ -33,7 +33,7 @@ class Reader
     public function __construct($source)
     {
         $this->source = $source;
-        $this->length = strlen($source);
+        $this->length = \strlen($source);
     }
 
     /**
diff --git a/Parser/Token.php b/Parser/Token.php
@@ -92,7 +92,7 @@ public function isDelimiter(array $values = array())
             return true;
         }
 
-        return in_array($this->value, $values);
+        return \in_array($this->value, $values);
     }
 
     /**
diff --git a/Parser/Tokenizer/TokenizerEscaping.php b/Parser/Tokenizer/TokenizerEscaping.php
@@ -65,13 +65,13 @@ private function replaceUnicodeSequences($value)
             $c = hexdec($match[1]);
 
             if (0x80 > $c %= 0x200000) {
-                return chr($c);
+                return \chr($c);
             }
             if (0x800 > $c) {
-                return chr(0xC0 | $c >> 6).chr(0x80 | $c & 0x3F);
+                return \chr(0xC0 | $c >> 6).\chr(0x80 | $c & 0x3F);
             }
             if (0x10000 > $c) {
-                return chr(0xE0 | $c >> 12).chr(0x80 | $c >> 6 & 0x3F).chr(0x80 | $c & 0x3F);
+                return \chr(0xE0 | $c >> 12).\chr(0x80 | $c >> 6 & 0x3F).\chr(0x80 | $c & 0x3F);
             }
         }, $value);
     }
diff --git a/Tests/Parser/Handler/AbstractHandlerTest.php b/Tests/Parser/Handler/AbstractHandlerTest.php
@@ -63,7 +63,7 @@ protected function assertRemainingContent(Reader $reader, $remainingContent)
             $this->assertEquals(0, $reader->getRemainingLength());
             $this->assertTrue($reader->isEOF());
         } else {
-            $this->assertEquals(strlen($remainingContent), $reader->getRemainingLength());
+            $this->assertEquals(\strlen($remainingContent), $reader->getRemainingLength());
             $this->assertEquals(0, $reader->getOffset($remainingContent));
         }
     }
diff --git a/Tests/XPath/TranslatorTest.php b/Tests/XPath/TranslatorTest.php
@@ -37,9 +37,9 @@ public function testXmlLang($css, array $elementsId)
         $translator = new Translator();
         $document = new \SimpleXMLElement(file_get_contents(__DIR__.'/Fixtures/lang.xml'));
         $elements = $document->xpath($translator->cssToXPath($css));
-        $this->assertCount(count($elementsId), $elements);
+        $this->assertCount(\count($elementsId), $elements);
         foreach ($elements as $element) {
-            $this->assertTrue(in_array($element->attributes()->id, $elementsId));
+            $this->assertTrue(\in_array($element->attributes()->id, $elementsId));
         }
     }
 
@@ -54,10 +54,10 @@ public function testHtmlIds($css, array $elementsId)
         $document->loadHTMLFile(__DIR__.'/Fixtures/ids.html');
         $document = simplexml_import_dom($document);
         $elements = $document->xpath($translator->cssToXPath($css));
-        $this->assertCount(count($elementsId), $elementsId);
+        $this->assertCount(\count($elementsId), $elementsId);
         foreach ($elements as $element) {
             if (null !== $element->attributes()->id) {
-                $this->assertTrue(in_array($element->attributes()->id, $elementsId));
+                $this->assertTrue(\in_array($element->attributes()->id, $elementsId));
             }
         }
         libxml_clear_errors();
diff --git a/XPath/Extension/AttributeMatchingExtension.php b/XPath/Extension/AttributeMatchingExtension.php
@@ -128,7 +128,7 @@ public function translateSuffixMatch(XPathExpr $xpath, $attribute, $value)
         return $xpath->addCondition($value ? sprintf(
             '%1$s and substring(%1$s, string-length(%1$s)-%2$s) = %3$s',
             $attribute,
-            strlen($value) - 1,
+            \strlen($value) - 1,
             Translator::getXpathLiteral($value)
         ) : '0');
     }
diff --git a/XPath/Translator.php b/XPath/Translator.php
@@ -176,7 +176,7 @@ public function nodeToXPath(NodeInterface $node)
             throw new ExpressionErrorException(sprintf('Node "%s" not supported.', $node->getNodeName()));
         }
 
-        return call_user_func($this->nodeTranslators[$node->getNodeName()], $node, $this);
+        return \call_user_func($this->nodeTranslators[$node->getNodeName()], $node, $this);
     }
 
     /**
@@ -194,7 +194,7 @@ public function addCombination($combiner, NodeInterface $xpath, NodeInterface $c
             throw new ExpressionErrorException(sprintf('Combiner "%s" not supported.', $combiner));
         }
 
-        return call_user_func($this->combinationTranslators[$combiner], $this->nodeToXPath($xpath), $this->nodeToXPath($combinedXpath));
+        return \call_user_func($this->combinationTranslators[$combiner], $this->nodeToXPath($xpath), $this->nodeToXPath($combinedXpath));
     }
 
     /**
@@ -208,7 +208,7 @@ public function addFunction(XPathExpr $xpath, FunctionNode $function)
             throw new ExpressionErrorException(sprintf('Function "%s" not supported.', $function->getName()));
         }
 
-        return call_user_func($this->functionTranslators[$function->getName()], $xpath, $function);
+        return \call_user_func($this->functionTranslators[$function->getName()], $xpath, $function);
     }
 
     /**
@@ -225,7 +225,7 @@ public function addPseudoClass(XPathExpr $xpath, $pseudoClass)
             throw new ExpressionErrorException(sprintf('Pseudo-class "%s" not supported.', $pseudoClass));
         }
 
-        return call_user_func($this->pseudoClassTranslators[$pseudoClass], $xpath);
+        return \call_user_func($this->pseudoClassTranslators[$pseudoClass], $xpath);
     }
 
     /**
@@ -244,7 +244,7 @@ public function addAttributeMatching(XPathExpr $xpath, $operator, $attribute, $v
             throw new ExpressionErrorException(sprintf('Attribute matcher operator "%s" not supported.', $operator));
         }
 
-        return call_user_func($this->attributeMatchingTranslators[$operator], $xpath, $attribute, $value);
+        return \call_user_func($this->attributeMatchingTranslators[$operator], $xpath, $attribute, $value);
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ abstract class AbstractNode implements NodeInterface`
`34`	`34`	`public function getNodeName()`
`35`	`35`	`{`
`36`	`36`	`if (null === $this->nodeName) {`
`37`		`- $this->nodeName = preg_replace('~.*\\\\([^\\\\]+)Node$~', '$1', get_called_class());`
	`37`	`+ $this->nodeName = preg_replace('~.*\\\\([^\\\\]+)Node$~', '$1', \get_called_class());`
`38`	`38`	`}`
`39`	`39`
`40`	`40`	`return $this->nodeName;`
Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ public function handle(Reader $reader, TokenStream $stream)`
`51`	`51`
`52`	`52`	`$value = $this->escaping->escapeUnicode($match[1]);`
`53`	`53`	`$stream->push(new Token(Token::TYPE_HASH, $value, $reader->getPosition()));`
`54`		`- $reader->moveForward(strlen($match[0]));`
	`54`	`+ $reader->moveForward(\strlen($match[0]));`
`55`	`55`
`56`	`56`	`return true;`
`57`	`57`	`}`
Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ public function handle(Reader $reader, TokenStream $stream)`
`47`	`47`	`}`
`48`	`48`
`49`	`49`	`$stream->push(new Token(Token::TYPE_NUMBER, $match[0], $reader->getPosition()));`
`50`		`- $reader->moveForward(strlen($match[0]));`
	`50`	`+ $reader->moveForward(\strlen($match[0]));`
`51`	`51`
`52`	`52`	`return true;`
`53`	`53`	`}`
Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ public function handle(Reader $reader, TokenStream $stream)`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`$stream->push(new Token(Token::TYPE_WHITESPACE, $match[0], $reader->getPosition()));`
`42`		`- $reader->moveForward(strlen($match[0]));`
	`42`	`+ $reader->moveForward(\strlen($match[0]));`
`43`	`43`
`44`	`44`	`return true;`
`45`	`45`	`}`
Original file line number	Diff line number	Diff line change
`@@ -169,7 +169,7 @@ private function parseSimpleSelector(TokenStream $stream, $insideNegation = fals`
`169`	`169`	`{`
`170`	`170`	`$stream->skipWhitespace();`
`171`	`171`
`172`		`- $selectorStart = count($stream->getUsed());`
	`172`	`+ $selectorStart = \count($stream->getUsed());`
`173`	`173`	`$result = $this->parseElementNode($stream);`
`174`	`174`	`$pseudoElement = null;`
`175`	`175`
`@@ -206,7 +206,7 @@ private function parseSimpleSelector(TokenStream $stream, $insideNegation = fals`
`206`	`206`	`}`
`207`	`207`
`208`	`208`	`$identifier = $stream->getNextIdentifier();`
`209`		`- if (in_array(strtolower($identifier), array('first-line', 'first-letter', 'before', 'after'))) {`
	`209`	`+ if (\in_array(strtolower($identifier), array('first-line', 'first-letter', 'before', 'after'))) {`
`210`	`210`	`// Special case: CSS 2.1 pseudo-elements can have a single ':'.`
`211`	`211`	`// Any new pseudo-element must have two.`
`212`	`212`	`$pseudoElement = $identifier;`
`@@ -272,7 +272,7 @@ private function parseSimpleSelector(TokenStream $stream, $insideNegation = fals`
`272`	`272`	`}`
`273`	`273`	`}`
`274`	`274`
`275`		`- if (count($stream->getUsed()) === $selectorStart) {`
	`275`	`+ if (\count($stream->getUsed()) === $selectorStart) {`
`276`	`276`	`throw SyntaxErrorException::unexpectedToken('selector', $stream->getPeek());`
`277`	`277`	`}`
`278`	`278`
Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ class Reader`
`33`	`33`	`public function __construct($source)`
`34`	`34`	`{`
`35`	`35`	`$this->source = $source;`
`36`		`- $this->length = strlen($source);`
	`36`	`+ $this->length = \strlen($source);`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -92,7 +92,7 @@ public function isDelimiter(array $values = array())`
`92`	`92`	`return true;`
`93`	`93`	`}`
`94`	`94`
`95`		`- return in_array($this->value, $values);`
	`95`	`+ return \in_array($this->value, $values);`
`96`	`96`	`}`
`97`	`97`
`98`	`98`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -65,13 +65,13 @@ private function replaceUnicodeSequences($value)`
`65`	`65`	`$c = hexdec($match[1]);`
`66`	`66`
`67`	`67`	`if (0x80 > $c %= 0x200000) {`
`68`		`- return chr($c);`
	`68`	`+ return \chr($c);`
`69`	`69`	`}`
`70`	`70`	`if (0x800 > $c) {`
`71`		`- return chr(0xC0 \| $c >> 6).chr(0x80 \| $c & 0x3F);`
	`71`	`+ return \chr(0xC0 \| $c >> 6).\chr(0x80 \| $c & 0x3F);`
`72`	`72`	`}`
`73`	`73`	`if (0x10000 > $c) {`
`74`		`- return chr(0xE0 \| $c >> 12).chr(0x80 \| $c >> 6 & 0x3F).chr(0x80 \| $c & 0x3F);`
	`74`	`+ return \chr(0xE0 \| $c >> 12).\chr(0x80 \| $c >> 6 & 0x3F).\chr(0x80 \| $c & 0x3F);`
`75`	`75`	`}`
`76`	`76`	`}, $value);`
`77`	`77`	`}`