symfony
diff --git a/‎src/Symfony/Component/DependencyInjection/ServiceLocator.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/ServiceLocator.php
+5-7Lines changed: 5 additions & 7 deletions b/‎src/Symfony/Component/DependencyInjection/ServiceLocator.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/ServiceLocator.php
+5-7Lines changed: 5 additions & 7 deletions
diff --git a/‎src/Symfony/Component/DependencyInjection/Tests/ServiceLocatorTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/ServiceLocatorTest.php
+9-4Lines changed: 9 additions & 4 deletions b/‎src/Symfony/Component/DependencyInjection/Tests/ServiceLocatorTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/ServiceLocatorTest.php
+9-4Lines changed: 9 additions & 4 deletions
@@ -22,7 +22,6 @@
 class ServiceLocator implements PsrContainerInterface
 {
     private $factories;
-    private $values = array();
 
     /**
      * @param callable[] $factories
@@ -53,13 +52,12 @@ public function get($id)
             throw new ServiceCircularReferenceException($id, array($id, $id));
         }
 
-        if (false !== $factory) {
-            $this->factories[$id] = true;
-            $this->values[$id] = $factory();
-            $this->factories[$id] = false;
+        $this->factories[$id] = true;
+        try {
+            return $factory();
+        } finally {
+            $this->factories[$id] = $factory;
         }
-
-        return $this->values[$id];
     }
 
     public function __invoke($id)
 
@@ -40,15 +40,20 @@ public function testGet()
         $this->assertSame('baz', $locator->get('bar'));
     }
 
-    public function testGetDoesNotExecuteTheSameCallableTwice()
+    public function testGetDoesNotMemoize()
     {
         $i = 0;
-        $locator = new ServiceLocator(array('foo' => function () use (&$i) { $i++; return 'bar'; }));
+        $locator = new ServiceLocator(array(
+            'foo' => function () use (&$i) {
+                ++$i;
+
+                return 'bar';
+            },
+        ));
 
         $this->assertSame('bar', $locator->get('foo'));
         $this->assertSame('bar', $locator->get('foo'));
-        $this->assertSame('bar', $locator->get('foo'));
-        $this->assertSame(1, $i);
+        $this->assertSame(2, $i);
     }
 
     /**