[CLEANUP] Drop unused local variables (#1581)

Author: oliverklee

src/CSSList/CSSBlockList.php

@@ -154,7 +154,6 @@ protected function getAllSelectors(?string $specificitySearch = null): array
                     }
                     $targetSpecificity = (int) $targetSpecificity;
                     $selectorSpecificity = $selector->getSpecificity();
-                    $comparatorMatched = false;
                     switch ($comparator) {
                         case '<=':
                             $comparatorMatched = $selectorSpecificity <= $targetSpecificity;

src/CSSList/CSSList.php

@@ -362,7 +362,7 @@ public function removeDeclarationBlockBySelector($selectors, bool $removeAll = f
         if (!\is_array($selectors)) {
             $selectors = \explode(',', $selectors);
         }
-        foreach ($selectors as $key => &$selector) {
+        foreach ($selectors as &$selector) {
             if (!($selector instanceof Selector)) {
                 if (!Selector::isValid($selector)) {
                     throw new UnexpectedTokenException(

src/Parsing/ParserState.php

@@ -119,7 +119,6 @@ public function parseIdentifier(bool $ignoreCase = true): string
         if ($result === null) {
             throw new UnexpectedTokenException('', $this->peek(5), 'identifier', $this->lineNumber);
         }
-        $character = null;
         while (!$this->isEnd() && ($character = $this->parseCharacter(true)) !== null) {
             if (preg_match('/[a-zA-Z0-9\\x{00A0}-\\x{FFFF}_-]/Sux', $character) !== 0) {
                 $result .= $character;

src/RuleSet/RuleSet.php

@@ -110,7 +110,6 @@ public function addDeclaration(Declaration $declarationToAdd, ?Declaration $sibl
         $position = \count($this->declarations[$propertyName]);
 
         if ($sibling !== null) {
-            $siblingIsInSet = false;
             $siblingPosition = \array_search($sibling, $this->declarations[$propertyName], true);
             if ($siblingPosition !== false) {
                 $siblingIsInSet = true;

src/Value/CSSString.php

@@ -56,7 +56,6 @@ public static function parse(ParserState $parserState): CSSString
             $parserState->consume($quote);
         }
         $result = '';
-        $content = null;
         if ($quote === null) {
             // Unquoted strings end in whitespace or with braces, brackets, parentheses
             while (preg_match('/[\\s{}()<>\\[\\]]/isu', $parserState->peek()) === 0) {

src/Value/Value.php

@@ -147,7 +147,6 @@ public static function parseIdentifierOrFunction(ParserState $parserState, bool
      */
     public static function parsePrimitiveValue(ParserState $parserState)
     {
-        $value = null;
         $parserState->consumeWhiteSpace();
         if (
             \is_numeric($parserState->peek())

tests/ParserTest.php

@@ -341,7 +341,6 @@ public function declarationGetters(): void
         $document = self::parsedStructureForFile('values');
         $declarationBlocks = $document->getAllDeclarationBlocks();
         $headerBlock = $declarationBlocks[0];
-        $bodyBlock = $declarationBlocks[1];
         $backgroundHeaderDeclarations = $headerBlock->getDeclarations('background-');
         self::assertCount(2, $backgroundHeaderDeclarations);
         self::assertSame('background-color', $backgroundHeaderDeclarations[0]->getPropertyName());
@@ -1128,7 +1127,7 @@ public function microsoftFilterStrictParsing(): void
     {
         $this->expectException(UnexpectedTokenException::class);
 
-        $document = self::parsedStructureForFile('ms-filter', Settings::create()->beStrict());
+        self::parsedStructureForFile('ms-filter', Settings::create()->beStrict());
     }
 
     /**

tests/Unit/RuleSet/DeclarationBlockTest.php

@@ -294,7 +294,7 @@ public function parseInStrictModeThrowsExceptionWithInvalidSelector(
 
         $parserState = new ParserState($selector . ' {}', Settings::create()->beStrict());
 
-        $subject = DeclarationBlock::parse($parserState);
+        DeclarationBlock::parse($parserState);
     }
 
     /**
@@ -479,7 +479,6 @@ public function setSelectorsSetsSingleSelectorProvidedAsString(string $selector)
 
         $subject->setSelectors($selector);
 
-        $result = $subject->getSelectors();
         self::assertSame([$selector], self::getSelectorsAsStrings($subject));
     }
 
@@ -500,7 +499,6 @@ public function setSelectorsSetsTwoCommaSeparatedSelectorsProvidedAsString(
 
         $subject->setSelectors($joinedSelectors);
 
-        $result = $subject->getSelectors();
         self::assertSame([$firstSelector, $secondSelector], self::getSelectorsAsStrings($subject));
     }