fix infinite loop for ml-string on first line (#44)

trobro · web-flow · commit f10139a0fa0a · 2025-04-09T17:57:27.000+02:00
diff --git a/src/HJSON/HJSONParser.php b/src/HJSON/HJSONParser.php
@@ -368,9 +368,8 @@ private function peek($offs)
     {
         $index = $this->at + $offs;
 
-        // Mimic old behavior with mb_substr
         if ($index < 0) {
-            $index = 0;
+            return null;
         }
         if ($index >= $this->textLengthChars) {
             return "";
diff --git a/tests/assets/failStr9a_test.hjson b/tests/assets/failStr9a_test.hjson
@@ -0,0 +1,4 @@
+[
+[
+=
+[[''''''
diff --git a/tests/assets/strings3_result.hjson b/tests/assets/strings3_result.hjson
@@ -0,0 +1 @@
+""
diff --git a/tests/assets/strings3_result.json b/tests/assets/strings3_result.json
@@ -0,0 +1 @@
+""
diff --git a/tests/assets/strings3_test.hjson b/tests/assets/strings3_test.hjson
@@ -0,0 +1 @@
+''''''

Original file line number	Diff line number	Diff line change
`@@ -368,9 +368,8 @@ private function peek($offs)`
`368`	`368`	`{`
`369`	`369`	`$index = $this->at + $offs;`
`370`	`370`
`371`		`- // Mimic old behavior with mb_substr`
`372`	`371`	`if ($index < 0) {`
`373`		`- $index = 0;`
	`372`	`+ return null;`
`374`	`373`	`}`
`375`	`374`	`if ($index >= $this->textLengthChars) {`
`376`	`375`	`return "";`
-Original file line number
+Diff line change
@@ @@ -0,0 +1,4 @@ @@
 +[
 +[
 +=
 +[[''''''