diff options
| author | Stefan Behnel <stefan_ml@behnel.de> | 2020-04-07 09:27:03 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-04-07 09:27:03 +0200 |
| commit | 59b0443133341b9fdb23909fecf9ed2d96904041 (patch) | |
| tree | 1fc9e8582e98cc7fdb6b486ea0059ae63270966d | |
| parent | f364d6dd54eb879171fdb98dd744cb571edd4d6a (diff) | |
| download | cython-59b0443133341b9fdb23909fecf9ed2d96904041.tar.gz | |
Avoid creating unused "__defaults__" getter functions for fused dispatch function. (GH-3491)
Closes GH-3481.
| -rw-r--r-- | Cython/Compiler/ExprNodes.py | 6 | ||||
| -rw-r--r-- | Cython/Compiler/FusedNode.py | 2 |
2 files changed, 4 insertions, 4 deletions
diff --git a/Cython/Compiler/ExprNodes.py b/Cython/Compiler/ExprNodes.py index 212dca068..9e94d0bd8 100644 --- a/Cython/Compiler/ExprNodes.py +++ b/Cython/Compiler/ExprNodes.py @@ -9279,7 +9279,8 @@ class PyCFunctionNode(ExprNode, ModuleNameMixin): value=arg.default) for arg in default_kwargs]) self.defaults_kwdict = defaults_kwdict.analyse_types(env) - else: + elif not self.specialized_cpdefs: + # Fused dispatch functions do not support (dynamic) default arguments, only the specialisations do. if default_args: defaults_tuple = DefaultsTupleNode( self.pos, default_args, self.defaults_struct) @@ -9440,8 +9441,7 @@ class PyCFunctionNode(ExprNode, ModuleNameMixin): if self.defaults_kwdict: code.putln('__Pyx_CyFunction_SetDefaultsKwDict(%s, %s);' % ( self.result(), self.defaults_kwdict.py_result())) - if def_node.defaults_getter and not self.specialized_cpdefs: - # Fused functions do not support dynamic defaults, only their specialisations can have them for now. + if def_node.defaults_getter: code.putln('__Pyx_CyFunction_SetDefaultsGetter(%s, %s);' % ( self.result(), def_node.defaults_getter.entry.pyfunc_cname)) if self.annotations_dict: diff --git a/Cython/Compiler/FusedNode.py b/Cython/Compiler/FusedNode.py index 42f15eea3..b45928e54 100644 --- a/Cython/Compiler/FusedNode.py +++ b/Cython/Compiler/FusedNode.py @@ -640,7 +640,7 @@ class FusedCFuncDefNode(StatListNode): pyx_code.put_chunk( u""" - def __pyx_fused_cpdef(signatures, args, kwargs, defaults, *, _fused_sigindex={}): + def __pyx_fused_cpdef(signatures, args, kwargs, defaults, _fused_sigindex={}): # FIXME: use a typed signature - currently fails badly because # default arguments inherit the types we specify here! |