fixed some/all of the dynamic binding

Author: kodonnell

astroid/inference.py renamed to astroid/inference.pyx

@@ -35,14 +35,15 @@
 from astroid.interpreter import dunder_lookup
 from astroid import protocols
 from astroid import util
+cimport cython
 
 
 MANAGER = manager.AstroidManager()
 
 
 # .infer method ###############################################################
 
-
+@cython.binding(True)
 def infer_end(self, context=None):
     """inference's end for node such as Module, ClassDef, FunctionDef,
     Const...
@@ -78,6 +79,7 @@ def _infer_sequence_helper(node, context=None):
 
 @decorators.raise_if_nothing_inferred
 @decorators.path_wrapper
+@cython.binding(True)
 def infer_sequence(self, context=None):
     if not any(isinstance(e, nodes.Starred) for e in self.elts):
         yield self
@@ -93,6 +95,7 @@ def infer_sequence(self, context=None):
 nodes.Set._infer = infer_sequence
 
 
+@cython.binding(True)
 def infer_map(self, context=None):
     if not any(isinstance(k, nodes.DictUnpack) for k, _ in self.items):
         yield self
@@ -171,6 +174,7 @@ def _higher_function_scope(node):
         return current.parent
     return None
 
+@cython.binding(True)
 def infer_name(self, context=None):
     """infer a Name: use name lookup rules"""
     frame, stmts = self.lookup(self.name)
@@ -190,14 +194,13 @@ def infer_name(self, context=None):
     return bases._infer_stmts(stmts, context, frame)
 
 # pylint: disable=no-value-for-parameter
-nodes.Name._infer = decorators.raise_if_nothing_inferred(
-    decorators.path_wrapper(infer_name)
-)
+nodes.Name._infer = decorators.raise_if_nothing_inferred(decorators.path_wrapper(infer_name))
 nodes.AssignName.infer_lhs = infer_name # won't work with a path wrapper
 
 
 @decorators.raise_if_nothing_inferred
 @decorators.path_wrapper
+@cython.binding(True)
 def infer_call(self, context=None):
     """infer a Call node by trying to guess what the function returns"""
     callcontext = context.clone()
@@ -226,6 +229,7 @@ def infer_call(self, context=None):
 
 @decorators.raise_if_nothing_inferred
 @decorators.path_wrapper
+@cython.binding(True)
 def infer_import(self, context=None, asname=True):
     """infer an Import node: return the imported module/object"""
     name = context.lookupname
@@ -247,6 +251,7 @@ def infer_import(self, context=None, asname=True):
 nodes.Import._infer = infer_import
 
 
+@cython.binding(True)
 def infer_name_module(self, name):
     context = contextmod.InferenceContext()
     context.lookupname = name
@@ -256,6 +261,7 @@ def infer_name_module(self, name):
 
 @decorators.raise_if_nothing_inferred
 @decorators.path_wrapper
+@cython.binding(True)
 def infer_import_from(self, context=None, asname=True):
     """infer a ImportFrom node: return the imported module/object"""
     name = context.lookupname
@@ -288,6 +294,7 @@ def infer_import_from(self, context=None, asname=True):
 
 
 @decorators.raise_if_nothing_inferred
+@cython.binding(True)
 def infer_attribute(self, context=None):
     """infer an Attribute node by using getattr on the associated object"""
     for owner in self.expr.infer(context):
@@ -325,6 +332,7 @@ def infer_attribute(self, context=None):
 
 @decorators.raise_if_nothing_inferred
 @decorators.path_wrapper
+@cython.binding(True)
 def infer_global(self, context=None):
     if context.lookupname is None:
         raise exceptions.InferenceError(node=self, context=context)
@@ -345,6 +353,7 @@ def infer_global(self, context=None):
 
 
 @decorators.raise_if_nothing_inferred
+@cython.binding(True)
 def infer_subscript(self, context=None):
     """Inference for subscripts
 
@@ -407,6 +416,7 @@ def infer_subscript(self, context=None):
 
 @decorators.raise_if_nothing_inferred
 @decorators.path_wrapper
+@cython.binding(True)
 def _infer_boolop(self, context=None):
     """Infer a boolean operation (and / or / not).
 
@@ -462,6 +472,7 @@ def _infer_boolop(self, context=None):
 
 # UnaryOp, BinOp and AugAssign inferences
 
+@cython.binding(True)
 def _filter_operation_errors(self, infer_callable, context, error):
     for result in infer_callable(self, context):
         if isinstance(result, error):
@@ -473,6 +484,7 @@ def _filter_operation_errors(self, infer_callable, context, error):
             yield result
 
 
+@cython.binding(True)
 def _infer_unaryop(self, context=None):
     """Infer what an UnaryOp should return when evaluated."""
     for operand in self.operand.infer(context):
@@ -529,6 +541,7 @@ def _infer_unaryop(self, context=None):
 
 @decorators.raise_if_nothing_inferred
 @decorators.path_wrapper
+@cython.binding(True)
 def infer_unaryop(self, context=None):
     """Infer what an UnaryOp should return when evaluated."""
     yield from _filter_operation_errors(self, _infer_unaryop, context,
@@ -711,6 +724,7 @@ def _infer_binary_operation(left, right, binary_opnode, context, flow_factory):
     yield util.BadBinaryOperationMessage(left_type, binary_opnode.op, right_type)
 
 
+@cython.binding(True)
 def _infer_binop(self, context):
     """Binary operation inference logic."""
     if context is None:
@@ -745,6 +759,7 @@ def _infer_binop(self, context):
 
 @decorators.yes_if_nothing_inferred
 @decorators.path_wrapper
+@cython.binding(True)
 def infer_binop(self, context=None):
     return _filter_operation_errors(self, _infer_binop, context,
                                     util.BadBinaryOperationMessage)
@@ -753,6 +768,7 @@ def infer_binop(self, context=None):
 nodes.BinOp._infer = infer_binop
 
 
+@cython.binding(True)
 def _infer_augassign(self, context=None):
     """Inference logic for augmented binary operations."""
     if context is None:
@@ -779,6 +795,7 @@ def _infer_augassign(self, context=None):
 
 @decorators.raise_if_nothing_inferred
 @decorators.path_wrapper
+@cython.binding(True)
 def infer_augassign(self, context=None):
     return _filter_operation_errors(self, _infer_augassign, context,
                                     util.BadBinaryOperationMessage)
@@ -791,6 +808,7 @@ def infer_augassign(self, context=None):
 
 @decorators.raise_if_nothing_inferred
 @decorators.path_wrapper
+@cython.binding(True)
 def infer_arguments(self, context=None):
     name = context.lookupname
     if name is None:
@@ -801,6 +819,7 @@ def infer_arguments(self, context=None):
 
 @decorators.raise_if_nothing_inferred
 @decorators.path_wrapper
+@cython.binding(True)
 def infer_assign(self, context=None):
     """infer a AssignName/AssignAttr: need to inspect the RHS part of the
     assign node
@@ -819,6 +838,7 @@ def infer_assign(self, context=None):
 
 @decorators.raise_if_nothing_inferred
 @decorators.path_wrapper
+@cython.binding(True)
 def infer_empty_node(self, context=None):
     if not self.has_underlying_object():
         yield util.Uninferable
@@ -832,12 +852,14 @@ def infer_empty_node(self, context=None):
 
 @decorators.raise_if_nothing_inferred
 @decorators.path_wrapper
+@cython.binding(True)
 def infer_index(self, context=None):
     return self.value.infer(context)
 nodes.Index._infer = infer_index
 
 # TODO: move directly into bases.Instance when the dependency hell
 # will be solved.
+@cython.binding(True)
 def instance_getitem(self, index, context=None):
     # Rewrap index to Const for this case
     new_context = contextmod.bind_context_to_node(context, self)

astroid/protocols.py renamed to astroid/protocols.pyx

@@ -23,6 +23,7 @@
 import collections
 import operator as operator_mod
 import sys
+cimport cython
 
 from astroid import Store
 from astroid import arguments
@@ -118,6 +119,7 @@ def _infer_unary_op(obj, op):
 
 
 @decorators.yes_if_nothing_inferred
+@cython.binding(True)
 def const_infer_binary_op(self, opnode, operator, other, context, _):
     not_implemented = nodes.Const(NotImplemented)
     if isinstance(other, nodes.Const):
@@ -141,6 +143,7 @@ def const_infer_binary_op(self, opnode, operator, other, context, _):
 nodes.Const.infer_binary_op = const_infer_binary_op
 
 
+@cython.binding(True)
 def _multiply_seq_by_int(self, opnode, other, context):
     node = self.__class__(parent=opnode)
     elts = []
@@ -167,6 +170,7 @@ def _filter_uninferable_nodes(elts, context):
 
 
 @decorators.yes_if_nothing_inferred
+@cython.binding(True)
 def tl_infer_binary_op(self, opnode, operator, other, context, method):
     not_implemented = nodes.Const(NotImplemented)
     if isinstance(other, self.__class__) and operator == '+':
@@ -195,6 +199,7 @@ def tl_infer_binary_op(self, opnode, operator, other, context, method):
 
 
 @decorators.yes_if_nothing_inferred
+@cython.binding(True)
 def instance_class_infer_binary_op(self, opnode, operator, other, context, method):
     return method.infer_call_result(self, context)
 
@@ -252,6 +257,7 @@ def _resolve_looppart(parts, asspath, context):
                     break
 
 @decorators.raise_if_nothing_inferred
+@cython.binding(True)
 def for_assigned_stmts(self, node=None, context=None, asspath=None):
     if isinstance(self, nodes.AsyncFor) or getattr(self, 'is_async', False):
         # Skip inferring of async code for now
@@ -270,6 +276,7 @@ def for_assigned_stmts(self, node=None, context=None, asspath=None):
 nodes.Comprehension.assigned_stmts = for_assigned_stmts
 
 
+@cython.binding(True)
 def sequence_assigned_stmts(self, node=None, context=None, asspath=None):
     if asspath is None:
         asspath = []
@@ -286,12 +293,14 @@ def sequence_assigned_stmts(self, node=None, context=None, asspath=None):
 nodes.Tuple.assigned_stmts = sequence_assigned_stmts
 nodes.List.assigned_stmts = sequence_assigned_stmts
 
+@cython.binding(True)
 def assend_assigned_stmts(self, node=None, context=None, asspath=None):
     return self.parent.assigned_stmts(node=self, context=context)
 nodes.AssignName.assigned_stmts = assend_assigned_stmts
 nodes.AssignAttr.assigned_stmts = assend_assigned_stmts
 
 
+@cython.binding(True)
 def _arguments_infer_argname(self, name, context):
     # arguments information may be missing, in which case we can't do anything
     # more
@@ -339,6 +348,7 @@ def _arguments_infer_argname(self, name, context):
         yield util.Uninferable
 
 
+@cython.binding(True)
 def arguments_assigned_stmts(self, node=None, context=None, asspath=None):
     if context.callcontext:
         # reset call context/name
@@ -353,6 +363,7 @@ def arguments_assigned_stmts(self, node=None, context=None, asspath=None):
 
 
 @decorators.raise_if_nothing_inferred
+@cython.binding(True)
 def assign_assigned_stmts(self, node=None, context=None, asspath=None):
     if not asspath:
         yield self.value
@@ -363,6 +374,7 @@ def assign_assigned_stmts(self, node=None, context=None, asspath=None):
                 assign_path=asspath, context=context)
 
 
+@cython.binding(True)
 def assign_annassigned_stmts(self, node=None, context=None, asspath=None):
     for inferred in assign_assigned_stmts(self, node, context, asspath):
         if inferred is None:
@@ -416,6 +428,7 @@ def _resolve_asspart(parts, asspath, context):
 
 
 @decorators.raise_if_nothing_inferred
+@cython.binding(True)
 def excepthandler_assigned_stmts(self, node=None, context=None, asspath=None):
     for assigned in node_classes.unpack_infer(self.type):
         if isinstance(assigned, nodes.ClassDef):
@@ -429,6 +442,7 @@ def excepthandler_assigned_stmts(self, node=None, context=None, asspath=None):
 nodes.ExceptHandler.assigned_stmts = excepthandler_assigned_stmts
 
 
+@cython.binding(True)
 def _infer_context_manager(self, mgr, context):
     try:
         inferred = next(mgr.infer(context=context))
@@ -480,6 +494,7 @@ def _infer_context_manager(self, mgr, context):
 
 
 @decorators.raise_if_nothing_inferred
+@cython.binding(True)
 def with_assigned_stmts(self, node=None, context=None, asspath=None):
     """Infer names and other nodes from a *with* statement.
 
@@ -542,6 +557,7 @@ def __enter__(self):
 
 
 @decorators.yes_if_nothing_inferred
+@cython.binding(True)
 def starred_assigned_stmts(self, node=None, context=None, asspath=None):
     """
     Arguments:

astroid/raw_building.pyx

@@ -23,6 +23,7 @@ import logging
 import os
 import sys
 import types
+cimport cython
 
 from astroid import bases
 from astroid import manager
@@ -73,7 +74,6 @@ def attach_dummy_node(node, name, runtime_object=_marker):
     enode.object = runtime_object
     _attach_local_node(node, enode, name)
 
-cimport cython
 @cython.binding(True)
 def _has_underlying_object(self):
     return self.object is not None and self.object is not _marker