classification
Title: Incorrect Call grammar in documentation
Type: Stage:
Components: Documentation Versions: Python 2.7
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: docs@python Nosy List: docs@python, mjpieters
Priority: normal Keywords: patch

Created on 2014-08-27 10:55 by mjpieters, last changed 2014-08-28 09:47 by mjpieters. This issue is now closed.

Files
File name Uploaded Description Edit
issue22288.patch mjpieters, 2014-08-27 12:47 Documentation correction
Messages (4)
msg225970 - (view) Author: Martijn Pieters (mjpieters) * Date: 2014-08-27 10:55
The changes for issue #3473 introduced a documentation bug. The Call expression grammar implies that f(*[1, 2], *[3, 4]) is allowed:

| "*" `expression` ["," "*" `expression`] ["," "**" `expression`]

I think Benjamin meant to use:

| "*" `expression` ["," `keyword_arguments `] ["," "**" `expression`]

instead, see the corresponding commit for Python 3.x: http://hg.python.org/cpython/diff/6abada05c313/Doc/reference/expressions.rst
msg225980 - (view) Author: Martijn Pieters (mjpieters) * Date: 2014-08-27 12:42
Proposed fix added in my fork.
msg225981 - (view) Author: Martijn Pieters (mjpieters) * Date: 2014-08-27 12:47
Sigh, patch creation fails against a remove repository; I guess this only works for the default branch.

Attached as a patch file instead.
msg226016 - (view) Author: Martijn Pieters (mjpieters) * Date: 2014-08-28 09:47
Fixed by revision 3ae399c6ecf6
History
Date User Action Args
2014-08-28 09:47:58mjpieterssetstatus: open -> closed
resolution: fixed
messages: + msg226016
2014-08-27 12:47:57mjpieterssethgrepos: - hgrepo270
2014-08-27 12:47:50mjpieterssetfiles: + issue22288.patch

messages: + msg225981
2014-08-27 12:45:07mjpieterssetfiles: - ffe77dc2979a.diff
2014-08-27 12:44:29mjpieterssetfiles: + ffe77dc2979a.diff
keywords: + patch
2014-08-27 12:42:13mjpieterssethgrepos: + hgrepo270
messages: + msg225980
2014-08-27 10:55:55mjpieterscreate