From: Robert Bradshaw <robertwb@math.washington.edu>
Date: Tue, 5 Aug 2008 07:57:30 +0000 (-0700)
Subject: Re-instate 908 with working type names as argument names.
X-Git-Tag: 0.9.8.1~93
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=0817b5c8f56265f0fb94c6634d4c1fed8f43a71a;p=cython.git

Re-instate 908 with working type names as argument names.
---

diff --git a/Cython/Compiler/Nodes.py b/Cython/Compiler/Nodes.py
index d06324c9..8290c739 100644
--- a/Cython/Compiler/Nodes.py
+++ b/Cython/Compiler/Nodes.py
@@ -362,8 +362,12 @@ class CNameDeclaratorNode(CDeclaratorNode):
     
     def analyse(self, base_type, env, nonempty = 0):
         if nonempty and self.name == '':
-            # Must have mistaken the name for the type. 
-            self.name = base_type.name
+            # May have mistaken the name for the type. 
+            if base_type.is_ptr or base_type.is_array or base_type.is_buffer:
+                error(self.pos, "Missing argument name")
+            elif base_type.is_void:
+                error(self.pos, "Use spam() rather than spam(void) to declare a function with no arguments.")
+            self.name = base_type.declaration_code("", for_display=1, pyrex=1)
             base_type = py_object_type
         self.type = base_type
         return self, base_type
@@ -422,6 +426,8 @@ class CFuncDeclaratorNode(CDeclaratorNode):
     optional_arg_count = 0
 
     def analyse(self, return_type, env, nonempty = 0):
+        if nonempty:
+            nonempty -= 1
         func_type_args = []
         for arg_node in self.args:
             name_declarator, type = arg_node.analyse(env, nonempty = nonempty)
@@ -1050,7 +1056,8 @@ class CFuncDefNode(FuncDefNode):
         
     def analyse_declarations(self, env):
         base_type = self.base_type.analyse(env)
-        name_declarator, type = self.declarator.analyse(base_type, env, self.body is not None)
+        # The 2 here is because we need both function and argument names. 
+        name_declarator, type = self.declarator.analyse(base_type, env, nonempty = 2 * (self.body is not None))
         if not type.is_cfunction:
             error(self.pos, 
                 "Suite attached to non-function declaration")
diff --git a/tests/compile/cargdef.pyx b/tests/compile/cargdef.pyx
index c8064d0b..c5da39bb 100644
--- a/tests/compile/cargdef.pyx
+++ b/tests/compile/cargdef.pyx
@@ -1,3 +1,10 @@
 def f(obj, int i, float f, char *s1, char s2[]):
     pass
-    
\ No newline at end of file
+    
+cdef g(obj, int i, float f, char *s1, char s2[]):
+    pass
+    
+cdef do_g(object (*func)(object, int, float, char*, char*)):
+    return func(1, 2, 3.14159, "a", "b")
+    
+do_g(&g)