Skip to content

Function definition grammar is incorrect #122845

@skirpichev

Description

@skirpichev

.. productionlist:: python-grammar
funcdef: [`decorators`] "def" `funcname` [`type_params`] "(" [`parameter_list`] ")"
: ["->" `expression`] ":" `suite`
decorators: `decorator`+
decorator: "@" `assignment_expression` NEWLINE
parameter_list: `defparameter` ("," `defparameter`)* "," "/" ["," [`parameter_list_no_posonly`]]
: | `parameter_list_no_posonly`
parameter_list_no_posonly: `defparameter` ("," `defparameter`)* ["," [`parameter_list_starargs`]]
: | `parameter_list_starargs`
parameter_list_starargs: "*" [`parameter`] ("," `defparameter`)* ["," ["**" `parameter` [","]]]
: | "**" `parameter` [","]
parameter: `identifier` [":" `expression`]
defparameter: `parameter` ["=" `expression`]
funcname: `identifier`

In parameter_list_starargs - everything can be optional, except for *, and that permits incorrect function definitions like def foo(*):

>>> def foo(*): pass
  File "<stdin>", line 1
    def foo(*): pass
            ^
SyntaxError: named arguments must follow bare *

See d.p.o thread https://discuss.python.org/t/56998. I'll provide a patch, based on suggested solution.

CC @norpadon

Linked PRs

Metadata

Metadata

Assignees

No one assigned

    Labels

    docsDocumentation in the Doc dir

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions