Avoid extra loop

Author: sebastianbergmann

src/Target/MapBuilder.php

@@ -160,11 +160,17 @@ public function build(Filter $filter, FileAnalyser $analyser): array
             }
 
             foreach ($this->parentClasses($classDetails, $class) as $parentClass) {
-                $classes[$class->namespacedName()] = array_merge_recursive(
+                $merged = array_merge_recursive(
                     $classes[$class->namespacedName()],
                     $classes[$parentClass->namespacedName()],
                 );
 
+                foreach ($merged as $mergedFile => $lines) {
+                    $merged[$mergedFile] = array_unique($lines);
+                }
+
+                $classes[$class->namespacedName()] = $merged;
+
                 if (isset($classesThatExtendClass[$parentClass->namespacedName()])) {
                     $this->process($classesThatExtendClass, $parentClass->namespacedName(), $class->file(), $class->startLine(), $class->endLine());
                 }
@@ -187,15 +193,6 @@ public function build(Filter $filter, FileAnalyser $analyser): array
             unset($classesThatExtendClass[$className]);
         }
 
-        /**
-         * @todo Avoid duplication and remove this loop
-         */
-        foreach ($classes as $className => $files) {
-            foreach (array_keys($files) as $file) {
-                $classes[$className][$file] = array_unique($classes[$className][$file]);
-            }
-        }
-
         return [
             'namespaces'                    => $namespaces,
             'traits'                        => $traits,