[TASK] Use native types for all constructor parameters

Part of #811

CHANGELOG.md

@@ -23,7 +23,7 @@ Please also have a look at our
 ### Changed
 
 - Use more native type declarations and strict mode
-  (#641, #772, #774, #778, #804)
+  (#641, #772, #774, #778, #804, #818)
 - Mark parsing-internal classes and methods as `@internal` (#674)
 - Block installations on unsupported higher PHP versions (#691)
 - Improve performance of `Value::parseValue` with many delimiters by refactoring

src/CSSList/AtRuleBlockList.php

@@ -22,12 +22,7 @@ class AtRuleBlockList extends CSSBlockList implements AtRule
      */
     private $sArgs;
 
-    /**
-     * @param string $type
-     * @param string $arguments
-     * @param int $lineNumber
-     */
-    public function __construct($type, $arguments = '', $lineNumber = 0)
+    public function __construct(string $type, string $arguments = '', int $lineNumber = 0)
     {
         parent::__construct($lineNumber);
         $this->type = $type;

src/CSSList/CSSBlockList.php

@@ -20,10 +20,7 @@
  */
 abstract class CSSBlockList extends CSSList
 {
-    /**
-     * @param int $lineNumber
-     */
-    public function __construct($lineNumber = 0)
+    public function __construct(int $lineNumber = 0)
     {
         parent::__construct($lineNumber);
     }

src/CSSList/CSSList.php

@@ -48,10 +48,7 @@ abstract class CSSList implements Renderable, Commentable
      */
     protected $iLineNo;
 
-    /**
-     * @param int $iLineNo
-     */
-    public function __construct($iLineNo = 0)
+    public function __construct(int $iLineNo = 0)
     {
         $this->comments = [];
         $this->aContents = [];

src/CSSList/Document.php

@@ -18,10 +18,7 @@
  */
 class Document extends CSSBlockList
 {
-    /**
-     * @param int $iLineNo
-     */
-    public function __construct($iLineNo = 0)
+    public function __construct(int $iLineNo = 0)
     {
         parent::__construct($iLineNo);
     }

src/CSSList/KeyFrame.php

@@ -19,10 +19,7 @@ class KeyFrame extends CSSList implements AtRule
      */
     private $animationName;
 
-    /**
-     * @param int $iLineNo
-     */
-    public function __construct($iLineNo = 0)
+    public function __construct(int $iLineNo = 0)
     {
         parent::__construct($iLineNo);
         $this->vendorKeyFrame = null;

src/Comment/Comment.php

@@ -19,11 +19,7 @@ class Comment implements Renderable
      */
     protected $commentText;
 
-    /**
-     * @param string $commentText
-     * @param int $lineNumber
-     */
-    public function __construct($commentText = '', $lineNumber = 0)
+    public function __construct(string $commentText = '', int $lineNumber = 0)
     {
         $this->commentText = $commentText;
         $this->lineNumber = $lineNumber;

src/Parser.php

@@ -20,10 +20,9 @@ class Parser
 
     /**
      * @param string $sText the complete CSS as text (i.e., usually the contents of a CSS file)
-     * @param Settings|null $oParserSettings
      * @param int $iLineNo the line number (starting from 1, not from 0)
      */
-    public function __construct($sText, ?Settings $oParserSettings = null, $iLineNo = 1)
+    public function __construct(string $sText, ?Settings $oParserSettings = null, int $iLineNo = 1)
     {
         if ($oParserSettings === null) {
             $oParserSettings = Settings::create();

src/Parsing/Anchor.php

@@ -19,10 +19,7 @@ class Anchor
      */
     private $oParserState;
 
-    /**
-     * @param int $iPosition
-     */
-    public function __construct($iPosition, ParserState $oParserState)
+    public function __construct(int $iPosition, ParserState $oParserState)
     {
         $this->iPosition = $iPosition;
         $this->oParserState = $oParserState;

src/Parsing/OutputException.php

@@ -9,11 +9,7 @@
  */
 final class OutputException extends SourceException
 {
-    /**
-     * @param string $sMessage
-     * @param int $iLineNo
-     */
-    public function __construct($sMessage, $iLineNo = 0)
+    public function __construct(string $sMessage, int $iLineNo = 0)
     {
         parent::__construct($sMessage, $iLineNo);
     }

src/Parsing/ParserState.php

@@ -58,9 +58,8 @@ class ParserState
 
     /**
      * @param string $sText the complete CSS as text (i.e., usually the contents of a CSS file)
-     * @param int $iLineNo
      */
-    public function __construct($sText, Settings $oParserSettings, $iLineNo = 1)
+    public function __construct(string $sText, Settings $oParserSettings, int $iLineNo = 1)
     {
         $this->oParserSettings = $oParserSettings;
         $this->sText = $sText;

src/Parsing/SourceException.php

@@ -11,11 +11,7 @@ class SourceException extends \Exception
      */
     private $iLineNo;
 
-    /**
-     * @param string $sMessage
-     * @param int $iLineNo
-     */
-    public function __construct($sMessage, $iLineNo = 0)
+    public function __construct(string $sMessage, int $iLineNo = 0)
     {
         $this->iLineNo = $iLineNo;
         if ($iLineNo !== 0) {

src/Parsing/UnexpectedTokenException.php

@@ -26,13 +26,7 @@ class UnexpectedTokenException extends SourceException
      */
     private $sMatchType;
 
-    /**
-     * @param string $sExpected
-     * @param string $sFound
-     * @param string $sMatchType
-     * @param int $iLineNo
-     */
-    public function __construct($sExpected, $sFound, $sMatchType = 'literal', $iLineNo = 0)
+    public function __construct(string $sExpected, string $sFound, string $sMatchType = 'literal', int $iLineNo = 0)
     {
         $this->sExpected = $sExpected;
         $this->sFound = $sFound;

src/Property/CSSNamespace.php

@@ -32,12 +32,7 @@ class CSSNamespace implements AtRule
      */
     protected $comments;
 
-    /**
-     * @param string $mUrl
-     * @param string|null $sPrefix
-     * @param int $iLineNo
-     */
-    public function __construct($mUrl, $sPrefix = null, $iLineNo = 0)
+    public function __construct(string $mUrl, ?string $sPrefix = null, int $iLineNo = 0)
     {
         $this->mUrl = $mUrl;
         $this->sPrefix = $sPrefix;

src/Property/Charset.php

@@ -33,11 +33,7 @@ class Charset implements AtRule
      */
     protected $comments;
 
-    /**
-     * @param CSSString $oCharset
-     * @param int $iLineNo
-     */
-    public function __construct(CSSString $oCharset, $iLineNo = 0)
+    public function __construct(CSSString $oCharset, int $iLineNo = 0)
     {
         $this->oCharset = $oCharset;
         $this->iLineNo = $iLineNo;

src/Property/Import.php

@@ -33,12 +33,7 @@ class Import implements AtRule
      */
     protected $comments;
 
-    /**
-     * @param URL $oLocation
-     * @param string $sMediaQuery
-     * @param int $iLineNo
-     */
-    public function __construct(URL $oLocation, $sMediaQuery, $iLineNo = 0)
+    public function __construct(URL $oLocation, string $sMediaQuery, int $iLineNo = 0)
     {
         $this->oLocation = $oLocation;
         $this->sMediaQuery = $sMediaQuery;

src/Property/Selector.php

@@ -84,11 +84,7 @@ public static function isValid($sSelector)
         return \preg_match(static::SELECTOR_VALIDATION_RX, $sSelector);
     }
 
-    /**
-     * @param string $sSelector
-     * @param bool $bCalculateSpecificity
-     */
-    public function __construct($sSelector, $bCalculateSpecificity = false)
+    public function __construct(string $sSelector, bool $bCalculateSpecificity = false)
     {
         $this->setSelector($sSelector);
         if ($bCalculateSpecificity) {

src/Rule/Rule.php

@@ -56,12 +56,7 @@ class Rule implements Renderable, Commentable
      */
     protected $comments;
 
-    /**
-     * @param string $sRule
-     * @param int $iLineNo
-     * @param int $iColNo
-     */
-    public function __construct($sRule, $iLineNo = 0, $iColNo = 0)
+    public function __construct(string $sRule, int $iLineNo = 0, int $iColNo = 0)
     {
         $this->sRule = $sRule;
         $this->mValue = null;

src/RuleSet/AtRuleSet.php

@@ -25,12 +25,7 @@ class AtRuleSet extends RuleSet implements AtRule
      */
     private $sArgs;
 
-    /**
-     * @param string $sType
-     * @param string $sArgs
-     * @param int $iLineNo
-     */
-    public function __construct($sType, $sArgs = '', $iLineNo = 0)
+    public function __construct(string $sType, string $sArgs = '', int $iLineNo = 0)
     {
         parent::__construct($iLineNo);
         $this->sType = $sType;

src/RuleSet/DeclarationBlock.php

@@ -35,10 +35,7 @@ class DeclarationBlock extends RuleSet
      */
     private $aSelectors;
 
-    /**
-     * @param int $iLineNo
-     */
-    public function __construct($iLineNo = 0)
+    public function __construct(int $iLineNo = 0)
     {
         parent::__construct($iLineNo);
         $this->aSelectors = [];

src/RuleSet/RuleSet.php

@@ -39,10 +39,7 @@ abstract class RuleSet implements Renderable, Commentable
      */
     protected $comments;
 
-    /**
-     * @param int $iLineNo
-     */
-    public function __construct($iLineNo = 0)
+    public function __construct(int $iLineNo = 0)
     {
         $this->aRules = [];
         $this->iLineNo = $iLineNo;

src/Value/CSSFunction.php

@@ -19,12 +19,9 @@ class CSSFunction extends ValueList
     protected $sName;
 
     /**
-     * @param string $sName
      * @param RuleValueList|array<array-key, Value|string> $aArguments
-     * @param string $sSeparator
-     * @param int $iLineNo
      */
-    public function __construct($sName, $aArguments, $sSeparator = ',', $iLineNo = 0)
+    public function __construct(string $sName, $aArguments, string $sSeparator = ',', int $iLineNo = 0)
     {
         if ($aArguments instanceof RuleValueList) {
             $sSeparator = $aArguments->getListSeparator();

src/Value/CSSString.php

@@ -22,11 +22,7 @@ class CSSString extends PrimitiveValue
      */
     private $sString;
 
-    /**
-     * @param string $sString
-     * @param int $iLineNo
-     */
-    public function __construct($sString, $iLineNo = 0)
+    public function __construct(string $sString, int $iLineNo = 0)
     {
         $this->sString = $sString;
         parent::__construct($iLineNo);

src/Value/CalcRuleValueList.php

@@ -8,10 +8,7 @@
 
 class CalcRuleValueList extends RuleValueList
 {
-    /**
-     * @param int $iLineNo
-     */
-    public function __construct($iLineNo = 0)
+    public function __construct(int $iLineNo = 0)
     {
         parent::__construct(',', $iLineNo);
     }

src/Value/Color.php

@@ -17,9 +17,8 @@ class Color extends CSSFunction
 {
     /**
      * @param array<array-key, Value|string> $colorValues
-     * @param int $lineNumber
      */
-    public function __construct(array $colorValues, $lineNumber = 0)
+    public function __construct(array $colorValues, int $lineNumber = 0)
     {
         parent::__construct(\implode('', \array_keys($colorValues)), $colorValues, ',', $lineNumber);
     }

src/Value/LineName.php

@@ -13,9 +13,8 @@ class LineName extends ValueList
 {
     /**
      * @param array<int, Value|string> $aComponents
-     * @param int $iLineNo
      */
-    public function __construct(array $aComponents = [], $iLineNo = 0)
+    public function __construct(array $aComponents = [], int $iLineNo = 0)
     {
         parent::__construct($aComponents, ' ', $iLineNo);
     }

src/Value/PrimitiveValue.php

@@ -6,10 +6,7 @@
 
 abstract class PrimitiveValue extends Value
 {
-    /**
-     * @param int $iLineNo
-     */
-    public function __construct($iLineNo = 0)
+    public function __construct(int $iLineNo = 0)
     {
         parent::__construct($iLineNo);
     }

src/Value/RuleValueList.php

@@ -11,11 +11,7 @@
  */
 class RuleValueList extends ValueList
 {
-    /**
-     * @param string $sSeparator
-     * @param int $iLineNo
-     */
-    public function __construct($sSeparator = ',', $iLineNo = 0)
+    public function __construct(string $sSeparator = ',', int $iLineNo = 0)
     {
         parent::__construct([], $sSeparator, $iLineNo);
     }

src/Value/Size.php

@@ -69,11 +69,8 @@ class Size extends PrimitiveValue
 
     /**
      * @param float|int|string $fSize
-     * @param string|null $sUnit
-     * @param bool $bIsColorComponent
-     * @param int $iLineNo
      */
-    public function __construct($fSize, $sUnit = null, $bIsColorComponent = false, $iLineNo = 0)
+    public function __construct($fSize, ?string $sUnit = null, bool $bIsColorComponent = false, int $iLineNo = 0)
     {
         parent::__construct($iLineNo);
         $this->fSize = (float) $fSize;

src/Value/URL.php

@@ -20,10 +20,7 @@ class URL extends PrimitiveValue
      */
     private $oURL;
 
-    /**
-     * @param int $iLineNo
-     */
-    public function __construct(CSSString $oURL, $iLineNo = 0)
+    public function __construct(CSSString $oURL, int $iLineNo = 0)
     {
         parent::__construct($iLineNo);
         $this->oURL = $oURL;

src/Value/Value.php

@@ -21,10 +21,7 @@ abstract class Value implements Renderable
      */
     protected $iLineNo;
 
-    /**
-     * @param int $iLineNo
-     */
-    public function __construct($iLineNo = 0)
+    public function __construct(int $iLineNo = 0)
     {
         $this->iLineNo = $iLineNo;
     }