Revert "Prevent nested lock acquisition in operations and function calls"

This reverts commit 778550c9.

Cython/Compiler/CypclassTransforms.py

@@ -512,27 +512,6 @@ class CypclassLockTransform(Visitor.EnvTransform):
         node.base = self.visit(node.base)
         return node
 
-    def visit_SimpleCallNode(self, node):
-        if node.type.is_error:
-            return node
-        for i, arg in enumerate(node.args or ()):
-            node.args[i] = self.visit_value(arg)
-        self.visitchildren(node, exclude=['args'])
-        return node
-
-    # Todo: other kinds of method calls
-
-    def visit_BinopNode(self, node):
-        node.operand1 = self.visit_value(node.operand1)
-        node.operand2 = self.visit_value(node.operand2)
-        return node
-
-    def visit_UnopNode(self, node):
-        node.operand = self.visit_value(node.operand)
-        return node
-
-    # Todo: other kinds of operations (comparisons, ...)
-
     def visit_DelStatNode(self, node):
         for arg in node.args:
             arg_entry = self.id(arg)

Cython/Compiler/ExprNodes.py

@@ -5935,8 +5935,7 @@ class SimpleCallNode(CallNode):
     #  explicit_cpp_self   bool            used internally
     #  needs_deref    bool                 used internally
 
-    # evaluate arguments first.
-    subexprs = ['self', 'coerced_self', 'args', 'arg_tuple', 'function']
+    subexprs = ['self', 'coerced_self', 'function', 'args', 'arg_tuple']
 
     self = None
     coerced_self = None