From: Stefan Behnel <scoder@users.berlios.de>
Date: Wed, 28 Jul 2010 09:44:40 +0000 (+0200)
Subject: code cleanup for function default arguments, make sure we keep any executable C code... 
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=e15cf1e1b60e3a0f8f4df73290426a325377a24e;p=cython.git

code cleanup for function default arguments, make sure we keep any executable C code behind the type declarations in the function header regardless of how we get to the default values
---

diff --git a/Cython/Compiler/Nodes.py b/Cython/Compiler/Nodes.py
index a02629f7..97a1b80a 100644
--- a/Cython/Compiler/Nodes.py
+++ b/Cython/Compiler/Nodes.py
@@ -2593,26 +2593,21 @@ class DefNode(FuncDefNode):
         all_args = tuple(positional_args) + tuple(kw_only_args)
         max_args = len(all_args)
 
-        default_args = []
-        for i, arg in enumerate(all_args):
-            if arg.default and arg.type.is_pyobject:
-                default_value = arg.calculate_default_value_code(code)
-                if arg.type is not PyrexTypes.py_object_type:
-                    default_value = "(PyObject*)"+default_value
-                default_args.append((i, default_value))
-
         code.putln("Py_ssize_t kw_args = PyDict_Size(%s);" %
                    Naming.kwds_cname)
         # the 'values' array collects borrowed references to arguments
         # before doing any type coercion etc.
         code.putln("PyObject* values[%d] = {%s};" % (
             max_args, ','.join('0'*max_args)))
-        # it looks funny to separate the init-to-0 from setting the
-        # default value, but C89 needs this
-        for i, default_value in default_args:
-            code.putln('values[%d] = %s;' % (i, default_value))
 
-        # parse the tuple and check that it's not too long
+        # assign borrowed Python default values to the values array,
+        # so that they can be overwritten by received arguments below
+        for i, arg in enumerate(all_args):
+            if arg.default and arg.type.is_pyobject:
+                default_value = arg.calculate_default_value_code(code)
+                code.putln('values[%d] = %s;' % (i, arg.type.as_pyobject(default_value)))
+
+        # parse the args tuple and check that it's not too long
         code.putln('switch (PyTuple_GET_SIZE(%s)) {' % Naming.args_cname)
         if self.star_arg:
             code.putln('default:')