[TASK] Use native types for all constructor parameters

Also make one condition more strict (via Rector).

Part of #811

Author: oliverklee

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);
     }