minor #17395 [DependencyInjection] Remove useless array declaration (paradajozsef)

Tobion · Tobion · commit 84623eae9d1c · 2016-01-16T00:34:46.000+01:00
This PR was merged into the 3.0 branch. Discussion ---------- [DependencyInjection] Remove useless array declaration | Q | A | ------------- | --- | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | - In [2.8](https://github.com/symfony/symfony/blob/2.8/src/Symfony/Component/DependencyInjection/Compiler/CheckReferenceValidityPass.php#L54) ```$ancestors``` was populated with ```$scopes```. 4a3e2f4 removed this, except the array declaration. Commits ------- 7240318 [3.0] Remove useless array declaration
diff --git a/src/Symfony/Component/DependencyInjection/Compiler/CheckReferenceValidityPass.php b/src/Symfony/Component/DependencyInjection/Compiler/CheckReferenceValidityPass.php
@@ -38,8 +38,6 @@ public function process(ContainerBuilder $container)
     {
         $this->container = $container;
 
-        $ancestors = array();
-
         foreach ($container->getDefinitions() as $id => $definition) {
             if ($definition->isSynthetic() || $definition->isAbstract()) {
                 continue;

Original file line number	Diff line number	Diff line change
`@@ -38,8 +38,6 @@ public function process(ContainerBuilder $container)`
`38`	`38`	`{`
`39`	`39`	`$this->container = $container;`
`40`	`40`
`41`		`- $ancestors = array();`
`42`		`-`
`43`	`41`	`foreach ($container->getDefinitions() as $id => $definition) {`
`44`	`42`	`if ($definition->isSynthetic() \|\| $definition->isAbstract()) {`
`45`	`43`	`continue;`