symfony
diff --git a/‎src/Symfony/Component/VarDumper/Caster/ReflectionCaster.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/VarDumper/Caster/ReflectionCaster.php
+26-11Lines changed: 26 additions & 11 deletions b/‎src/Symfony/Component/VarDumper/Caster/ReflectionCaster.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/VarDumper/Caster/ReflectionCaster.php
+26-11Lines changed: 26 additions & 11 deletions
diff --git a/‎src/Symfony/Component/VarDumper/Tests/Caster/ReflectionCasterTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/VarDumper/Tests/Caster/ReflectionCasterTest.php
+21-8Lines changed: 21 additions & 8 deletions b/‎src/Symfony/Component/VarDumper/Tests/Caster/ReflectionCasterTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/VarDumper/Tests/Caster/ReflectionCasterTest.php
+21-8Lines changed: 21 additions & 8 deletions
@@ -76,7 +76,20 @@ public static function castClosure(\Closure $c, array $a, Stub $stub, $isNested)
 
     public static function castGenerator(\Generator $c, array $a, Stub $stub, $isNested)
     {
-        return class_exists('ReflectionGenerator', false) ? self::castReflectionGenerator(new \ReflectionGenerator($c), $a, $stub, $isNested) : $a;
+        if (!class_exists('ReflectionGenerator', false)) {
+            return $a;
+        }
+
+        // Cannot create ReflectionGenerator based on a terminated Generator
+        try {
+            $reflectionGenerator = new \ReflectionGenerator($c);
+        } catch (\Exception $e) {
+            $a[Caster::PREFIX_VIRTUAL.'closed'] = true;
+
+            return $a;
+        }
+
+        return self::castReflectionGenerator($reflectionGenerator, $a, $stub, $isNested);
     }
 
     public static function castType(\ReflectionType $c, array $a, Stub $stub, $isNested)
@@ -99,31 +112,33 @@ public static function castReflectionGenerator(\ReflectionGenerator $c, array $a
         if ($c->getThis()) {
             $a[$prefix.'this'] = new CutStub($c->getThis());
         }
-        $x = $c->getFunction();
+        $function = $c->getFunction();
         $frame = array(
-            'class' => isset($x->class) ? $x->class : null,
-            'type' => isset($x->class) ? ($x->isStatic() ? '::' : '->') : null,
-            'function' => $x->name,
+            'class' => isset($function->class) ? $function->class : null,
+            'type' => isset($function->class) ? ($function->isStatic() ? '::' : '->') : null,
+            'function' => $function->name,
             'file' => $c->getExecutingFile(),
             'line' => $c->getExecutingLine(),
         );
         if ($trace = $c->getTrace(DEBUG_BACKTRACE_IGNORE_ARGS)) {
-            $x = new \ReflectionGenerator($c->getExecutingGenerator());
+            $function = new \ReflectionGenerator($c->getExecutingGenerator());
             array_unshift($trace, array(
                 'function' => 'yield',
-                'file' => $x->getExecutingFile(),
-                'line' => $x->getExecutingLine() - 1,
+                'file' => $function->getExecutingFile(),
+                'line' => $function->getExecutingLine() - 1,
             ));
             $trace[] = $frame;
             $a[$prefix.'trace'] = new TraceStub($trace, false, 0, -1, -1);
         } else {
-            $x = new FrameStub($frame, false, true);
-            $x = ExceptionCaster::castFrameStub($x, array(), $x, true);
+            $function = new FrameStub($frame, false, true);
+            $function = ExceptionCaster::castFrameStub($function, array(), $function, true);
             $a[$prefix.'executing'] = new EnumStub(array(
-                $frame['class'].$frame['type'].$frame['function'].'()' => $x[$prefix.'src'],
+                $frame['class'].$frame['type'].$frame['function'].'()' => $function[$prefix.'src'],
             ));
         }
 
+        $a[Caster::PREFIX_VIRTUAL.'closed'] = false;
+
         return $a;
     }
 
 
@@ -149,11 +149,10 @@ public function testGenerator()
             $this->markTestSkipped('xdebug is active');
         }
 
-        $g = new GeneratorDemo();
-        $g = $g->baz();
-        $r = new \ReflectionGenerator($g);
+        $generator = new GeneratorDemo();
+        $generator = $generator->baz();
 
-        $xDump = <<<'EODUMP'
+        $expectedDump = <<<'EODUMP'
 Generator {
   this: Symfony\Component\VarDumper\Tests\Fixtures\GeneratorDemo { …}
   executing: {
@@ -165,16 +164,17 @@ public function testGenerator()
         """
     }
   }
+  closed: false
 }
 EODUMP;
 
-        $this->assertDumpMatchesFormat($xDump, $g);
+        $this->assertDumpMatchesFormat($expectedDump, $generator);
 
-        foreach ($g as $v) {
+        foreach ($generator as $v) {
             break;
         }
 
-        $xDump = <<<'EODUMP'
+        $expectedDump = <<<'EODUMP'
 array:2 [
   0 => ReflectionGenerator {
     this: Symfony\Component\VarDumper\Tests\Fixtures\GeneratorDemo { …}
@@ -207,6 +207,7 @@ public function testGenerator()
         }
       }
     }
+    closed: false
   }
   1 => Generator {
     executing: {
@@ -218,11 +219,23 @@ public function testGenerator()
           """
       }
     }
+    closed: false
   }
 ]
 EODUMP;
 
-        $this->assertDumpMatchesFormat($xDump, array($r, $r->getExecutingGenerator()));
+        $r = new \ReflectionGenerator($generator);
+        $this->assertDumpMatchesFormat($expectedDump, array($r, $r->getExecutingGenerator()));
+
+        foreach ($generator as $v) {
+        }
+
+        $expectedDump = <<<'EODUMP'
+Generator {
+  closed: true
+}
+EODUMP;
+        $this->assertDumpMatchesFormat($expectedDump, $generator);
     }
 }
Original file line number	Diff line number	Diff line change
`@@ -149,11 +149,10 @@ public function testGenerator()`
`149`	`149`	`$this->markTestSkipped('xdebug is active');`
`150`	`150`	`}`
`151`	`151`
`152`		`- $g = new GeneratorDemo();`
`153`		`- $g = $g->baz();`
`154`		`- $r = new \ReflectionGenerator($g);`
	`152`	`+ $generator = new GeneratorDemo();`
	`153`	`+ $generator = $generator->baz();`
`155`	`154`
`156`		`- $xDump = <<<'EODUMP'`
	`155`	`+ $expectedDump = <<<'EODUMP'`
`157`	`156`	`Generator {`
`158`	`157`	`this: Symfony\Component\VarDumper\Tests\Fixtures\GeneratorDemo { …}`
`159`	`158`	`executing: {`
`@@ -165,16 +164,17 @@ public function testGenerator()`
`165`	`164`	`"""`
`166`	`165`	`}`
`167`	`166`	`}`
	`167`	`+ closed: false`
`168`	`168`	`}`
`169`	`169`	`EODUMP;`
`170`	`170`
`171`		`- $this->assertDumpMatchesFormat($xDump, $g);`
	`171`	`+ $this->assertDumpMatchesFormat($expectedDump, $generator);`
`172`	`172`
`173`		`- foreach ($g as $v) {`
	`173`	`+ foreach ($generator as $v) {`
`174`	`174`	`break;`
`175`	`175`	`}`
`176`	`176`
`177`		`- $xDump = <<<'EODUMP'`
	`177`	`+ $expectedDump = <<<'EODUMP'`
`178`	`178`	`array:2 [`
`179`	`179`	`0 => ReflectionGenerator {`
`180`	`180`	`this: Symfony\Component\VarDumper\Tests\Fixtures\GeneratorDemo { …}`
`@@ -207,6 +207,7 @@ public function testGenerator()`
`207`	`207`	`}`
`208`	`208`	`}`
`209`	`209`	`}`
	`210`	`+ closed: false`
`210`	`211`	`}`
`211`	`212`	`1 => Generator {`
`212`	`213`	`executing: {`
`@@ -218,11 +219,23 @@ public function testGenerator()`
`218`	`219`	`"""`
`219`	`220`	`}`
`220`	`221`	`}`
	`222`	`+ closed: false`
`221`	`223`	`}`
`222`	`224`	`]`
`223`	`225`	`EODUMP;`
`224`	`226`
`225`		`- $this->assertDumpMatchesFormat($xDump, array($r, $r->getExecutingGenerator()));`
	`227`	`+ $r = new \ReflectionGenerator($generator);`
	`228`	`+ $this->assertDumpMatchesFormat($expectedDump, array($r, $r->getExecutingGenerator()));`
	`229`	`+`
	`230`	`+ foreach ($generator as $v) {`
	`231`	`+ }`
	`232`	`+`
	`233`	`+ $expectedDump = <<<'EODUMP'`
	`234`	`+Generator {`
	`235`	`+ closed: true`
	`236`	`+}`
	`237`	`+EODUMP;`
	`238`	`+ $this->assertDumpMatchesFormat($expectedDump, $generator);`
`226`	`239`	`}`
`227`	`240`	`}`
`228`	`241`