Unset *outside* the loops (performance)

Author: erayd

src/JsonSchema/Constraints/CollectionConstraint.php

@@ -83,9 +83,9 @@ protected function validateItems(&$value, $schema = null, JsonPointer $path = nu
                     $typeValidator->check($v, $schema->items, $k_path, $i);
 
                     $validator->check($v, $schema->items, $k_path, $i);
-                    unset($v); // remove dangling reference to prevent any future bugs
-                               // caused by accidentally using $v elsewhere
                 }
+                unset($v); // remove dangling reference to prevent any future bugs
+                           // caused by accidentally using $v elsewhere
                 $this->addErrors($typeValidator->getErrors());
                 $this->addErrors($validator->getErrors());
             } else {
@@ -107,9 +107,9 @@ protected function validateItems(&$value, $schema = null, JsonPointer $path = nu
                     } elseif (isset($secondErrors) && count($secondErrors) === count($this->getErrors())) {
                         $this->errors = $initErrors;
                     }
-                    unset($v); // remove dangling reference to prevent any future bugs
-                               // caused by accidentally using $v elsewhere
                 }
+                unset($v); // remove dangling reference to prevent any future bugs
+                           // caused by accidentally using $v elsewhere
             }
         } else {
             // Defined item type definitions
@@ -130,9 +130,9 @@ protected function validateItems(&$value, $schema = null, JsonPointer $path = nu
                         $this->checkUndefined($v, new \stdClass(), $path, $k);
                     }
                 }
-                unset($v); // remove dangling reference to prevent any future bugs
-                           // caused by accidentally using $v elsewhere
             }
+            unset($v); // remove dangling reference to prevent any future bugs
+                       // caused by accidentally using $v elsewhere
 
             // Treat when we have more schema definitions than values, not for empty arrays
             if (count($value) > 0) {