From: Robert Bradshaw <robertwb@math.washington.edu>
Date: Fri, 25 Feb 2011 06:04:26 +0000 (-0800)
Subject: One more side effect we want to avoid in argument ordering.
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=1f3995a3ae25d72ba91b072abc22e51af1a70451;p=cython.git

One more side effect we want to avoid in argument ordering.
---

diff --git a/Cython/Compiler/ExprNodes.py b/Cython/Compiler/ExprNodes.py
index 1772e6d9..c1d56d9b 100755
--- a/Cython/Compiler/ExprNodes.py
+++ b/Cython/Compiler/ExprNodes.py
@@ -3043,6 +3043,9 @@ class SimpleCallNode(CallNode):
                     or arg.entry.type.is_cfunction):
                     # local variables and C functions are safe
                     pass
+                elif arg.type.is_cpp_class:
+                    # Assignment has side effects, avoid.
+                    pass
                 elif env.nogil and arg.type.is_pyobject:
                     # can't copy a Python reference into a temp in nogil
                     # env (this is safe: a construction would fail in