[CLEANUP] Don't store array length as a property

Since PHP 5, if not earlier, the length of an array is internally stored by PHP
and can be accessed in O(1) time.
Maintaining both the array and its length in class properties is error-prone
(particularly regarding possible future code changes).
When used in a loop, `\count($array)` is more repetitively expensive than
`$arrayLength`,
but the latter can be set up as a local variable as and when needed.

Reference:
https://stackoverflow.com/questions/5835241/is-phps-count-function-o1-or-on-for-arrays

This change also ensures that the characters are always a (dazzling) array.
And adds a type parameter to `ParserState::setCharset()`,
which was unavoidably required.

Resolves #953.

Author: JakeQZ

src/Parsing/ParserState.php

@@ -30,7 +30,7 @@ class ParserState
     /**
      * @var array<int, string>
      */
-    private $characters;
+    private $characters = [];
 
     /**
      * @var int
@@ -44,11 +44,6 @@ class ParserState
      */
     private $charset;
 
-    /**
-     * @var int
-     */
-    private $length;
-
     /**
      * @var int
      */
@@ -75,7 +70,6 @@ public function setCharset(string $charset): void
     {
         $this->charset = $charset;
         $this->characters = $this->strsplit($this->text);
-        $this->length = \count($this->characters);
     }
 
     /**
@@ -225,7 +219,7 @@ public function consumeWhiteSpace(): array
                 try {
                     $oComment = $this->consumeComment();
                 } catch (UnexpectedEOFException $e) {
-                    $this->currentPosition = $this->length;
+                    $this->currentPosition = \count($this->characters);
                     return $comments;
                 }
             } else {
@@ -256,8 +250,13 @@ public function comes($sString, $bCaseInsensitive = false): bool
      */
     public function peek($length = 1, $offset = 0): string
     {
+<<<<<<< HEAD
         $offset += $this->currentPosition;
         if ($offset >= $this->length) {
+=======
+        $iOffset += $this->currentPosition;
+        if ($iOffset >= \count($this->characters)) {
+>>>>>>> 521415c ([CLEANUP] Don't store array length as a property)
             return '';
         }
         return $this->substr($offset, $length);
@@ -286,7 +285,7 @@ public function consume($mValue = 1): string
             $this->currentPosition += $this->strlen($mValue);
             return $mValue;
         } else {
-            if ($this->currentPosition + $mValue > $this->length) {
+            if ($this->currentPosition + $mValue > \count($this->characters)) {
                 throw new UnexpectedEOFException((string) $mValue, $this->peek(5), 'count', $this->lineNumber);
             }
             $result = $this->substr($this->currentPosition, $mValue);
@@ -343,7 +342,7 @@ public function consumeComment()
 
     public function isEnd(): bool
     {
-        return $this->currentPosition >= $this->length;
+        return $this->currentPosition >= \count($this->characters);
     }
 
     /**
@@ -436,10 +435,10 @@ public function strlen($sString): int
     private function substr($iStart, $length): string
     {
         if ($length < 0) {
-            $length = $this->length - $iStart + $length;
+            $length = \count($this->characters) - $iStart + $length;
         }
-        if ($iStart + $length > $this->length) {
-            $length = $this->length - $iStart;
+        if ($iStart + $length > \count($this->characters)) {
+            $length = \count($this->characters) - $iStart;
         }
         $result = '';
         while ($length > 0) {