Remove recursive calls (#961)

kubawerlos · Feb 3, 2024 · a80cd2b · a80cd2b
1 parent 60cb712
commit a80cd2b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 13 deletions.
diff --git a/src/Fixer/PhpdocArrayStyleFixer.php b/src/Fixer/PhpdocArrayStyleFixer.php
@@ -46,12 +46,10 @@ public function getPriority(): int
 
     protected function fixType(string $type): string
     {
-        $newType = Preg::replace('/([\\\\a-zA-Z0-9_>]+)\[\]/', 'array<$1>', $type);
+        do {
+            $type = Preg::replace('/([\\\\a-zA-Z0-9_>]+)\[\]/', 'array<$1>', $type, -1, $count);
+        } while ($count > 0);
 
-        if ($newType === $type) {
-            return $type;
-        }
-
-        return $this->fixType($newType);
+        return $type;
     }
 }
diff --git a/src/Fixer/PhpdocTypesCommaSpacesFixer.php b/src/Fixer/PhpdocTypesCommaSpacesFixer.php
@@ -34,12 +34,6 @@ public function getPriority(): int
 
     protected function fixType(string $type): string
     {
-        $newType = Preg::replace('/\h*,\s*/', ', ', $type);
-
-        if ($newType === $type) {
-            return $type;
-        }
-
-        return $this->fixType($newType);
+        return Preg::replace('/\h*,\s*/', ', ', $type);
     }
 }