Merge pull request #84 from samsonasik/improve-performance

Improve performance : count() and array_push removal when possible

Author: samsonasik

src/ConfigPostProcessor.php

@@ -5,8 +5,6 @@
 use function array_intersect_key;
 use function array_key_exists;
 use function array_pop;
-use function array_push;
-use function count;
 use function in_array;
 use function is_array;
 use function is_callable;
@@ -70,7 +68,7 @@ function ($value) {
             function ($value, array $keys) {
                 $key = array_pop($keys);
                 // Only worried about a top-level "router" key.
-                return $key === 'router' && count($keys) === 0 && is_array($value)
+                return $key === 'router' && $keys === [] && is_array($value)
                     ? [$this, 'noopReplacement']
                     : null;
             },
@@ -84,7 +82,7 @@ function ($value) {
 
             // Array values
             function ($value, array $keys) {
-                return 0 !== count($keys) && is_array($value)
+                return $keys !== [] && is_array($value)
                     ? [$this, '__invoke']
                     : null;
             },
@@ -153,7 +151,7 @@ private function replace($value, array $keys, $key = null)
     {
         // Add new key to the list of keys.
         // We do not need to remove it later, as we are working on a copy of the array.
-        array_push($keys, $key);
+        $keys[] = $key;
 
         // Identify rewrite strategy and perform replacements
         $rewriteRule = $this->replacementRuleMatch($value, $keys);