Inline cdef class method support, see ticket #474

Cython/Compiler/ParseTreeTransforms.py

@@ -567,7 +567,8 @@ class PxdPostParse(CythonTransform, SkipDeclarations):
             err = None # allow these slots
 
         if isinstance(node, Nodes.CFuncDefNode):
-            if u'inline' in node.modifiers and self.scope_type == 'pxd':
+            if (u'inline' in node.modifiers and
+                self.scope_type in ('pxd', 'cclass')):
                 node.inline_in_pxd = True
                 if node.visibility != 'private':
                     err = self.ERR_NOGO_WITH_INLINE % node.visibility

Cython/Compiler/Symtab.py

@@ -71,6 +71,7 @@ class Entry(object):
     # is_builtin_cmethod boolean  Is a C method of a builtin type (implies is_cmethod)
     # is_unbound_cmethod boolean  Is an unbound C method of an extension type
     # is_final_cmethod   boolean  Is non-overridable C method
+    # is_inline_cmethod  boolean  Is inlined C method
     # is_anonymous     boolean    Is a anonymous pyfunction entry
     # is_type          boolean    Is a type definition
     # is_cclass        boolean    Is an extension class
@@ -141,6 +142,7 @@ class Entry(object):
     is_builtin_cmethod = False
     is_unbound_cmethod = 0
     is_final_cmethod = 0
+    is_inline_cmethod = 0
     is_anonymous = 0
     is_type = 0
     is_cclass = 0
@@ -1821,7 +1823,9 @@ class CClassScope(ClassScope):
         if defining:
             entry.func_cname = self.mangle(Naming.func_prefix, name)
         entry.utility_code = utility_code
-        if self.directives.get('final'):
+        if u'inline' in modifiers:
+            entry.is_inline_cmethod = True
+        if self.directives.get('final') or entry.is_inline_cmethod:
             entry.is_final_cmethod = True
             entry.final_func_cname = entry.func_cname
         return entry
@@ -1884,7 +1888,9 @@ class CClassScope(ClassScope):
             entry.is_inherited = 1
             if base_entry.is_final_cmethod:
                 entry.is_final_cmethod = True
-                if self.parent_scope == base_scope.parent_scope:
+                entry.is_inline_cmethod = base_entry.is_inline_cmethod
+                if (self.parent_scope == base_scope.parent_scope or
+                    entry.is_inline_cmethod):
                     entry.final_func_cname = base_entry.final_func_cname
             if is_builtin:
                 entry.is_builtin_cmethod = True

tests/run/cmethod_inline_T474.pxd

+cdef class TestInlineMethod(object):
+    cdef inline int cdef_inline_method(self):
+        return 0

tests/run/cmethod_inline_T474.pyx

+# mode: run
+# ticket: 474
+cimport cython
+
+
+cdef class TestInlineMethod(object):
+    """
+    >>> test = TestInlineMethod()
+    >>> test.test_cdef_method()
+    0
+    """
+
+    @cython.test_assert_path_exists("//AttributeNode[@entry.is_final_cmethod=True]")
+    @cython.test_assert_path_exists("//AttributeNode[@entry.is_inline_cmethod=True]")
+    def test_cdef_method(self):
+        return self.cdef_inline_method()
+
+
+cdef class Subtyping(TestInlineMethod):
+    """
+    >>> test = Subtyping()
+    >>> test.test_cdef_subtyping()
+    0
+    """
+
+    @cython.test_assert_path_exists("//AttributeNode[@entry.is_final_cmethod=True]")
+    @cython.test_assert_path_exists("//AttributeNode[@entry.is_inline_cmethod=True]")
+    def test_cdef_subtyping(self):
+        return self.cdef_inline_method()