Don't raise StopIteration in InferenceContext.push, instead exit from the generator by doing an early return

This makes astroid compliant with PEP 479 changes, since in the future it will be
prohibited to exit from a generator by raising a StopIteration.

1 files changed, 3 insertions, 2 deletions

diff --git a/astroid/context.py b/astroid/context.py
index bb2422ae..5a2ea525 100644
--- a/astroid/context.py
+++ b/astroid/context.py
@@ -17,7 +17,6 @@
 # with astroid. If not, see <http://www.gnu.org/licenses/>.
 
 """Various context related utilities, including inference and call contexts."""
-
 import contextlib
 import itertools
 
@@ -38,8 +37,10 @@ class InferenceContext(object):
     def push(self, node):
         name = self.lookupname
         if (node, name) in self.path:
-            raise StopIteration()
+            return True
+
         self.path.add((node, name))
+        return False
 
     def clone(self):
         # XXX copy lookupname/callcontext ?