[issue16556] Update string.Formatter.vformat documentation to say "**kwargs"

2012-12-02 Thread Taavi Burns
Taavi Burns added the comment: Gak, sorry, didn't mean to change the status and resolution! -- resolution: -> fixed status: open -> closed ___ Python tracker ___ ___

[issue16556] Update string.Formatter.vformat documentation to say "**kwargs"

2012-12-02 Thread Taavi Burns
Taavi Burns added the comment: Ah, yes, I could have been more clear that I wasn't even looking at the source code itself. :) Thanks! -- resolution: fixed -> status: closed -> open versions: -Python 3.2, Python 3.4 ___ Python tracker

[issue16556] Update string.Formatter.vformat documentation to say "**kwargs"

2012-11-27 Thread Éric Araujo
Éric Araujo added the comment: Oh, I had totally missed that the mismatch was between the function signature in the docs and another part of the doc (I thought it was between the real function signature and the doc). Good catch! -- ___ Python track

[issue16556] Update string.Formatter.vformat documentation to say "**kwargs"

2012-11-27 Thread Ezio Melotti
Ezio Melotti added the comment: The patch looked OK, so I committed it. Regarding similar cleanups I think it's OK to do them only if the inconsistency is in the same function. If different functions use kwds or kwargs in different places but they do it consistently, then it doesn't matter. -

[issue16556] Update string.Formatter.vformat documentation to say "**kwargs"

2012-11-27 Thread Roundup Robot
Roundup Robot added the comment: New changeset d805982336a0 by Ezio Melotti in branch '2.7': #16556: Fix inconsistency between kwds and kwargs. Patch by Taavi Burns. http://hg.python.org/cpython/rev/d805982336a0 New changeset 565c3bbed7d3 by Ezio Melotti in branch '3.2': #16556: Fix inconsisten

[issue16556] Update string.Formatter.vformat documentation to say "**kwargs"

2012-11-25 Thread Taavi Burns
Taavi Burns added the comment: The issue of churn is totally fair. I'd still advocate for the attached patch, as it's tiny and fixes an inconsistency between the function signature and the docs that are referring to that signature. Thanks! -- ___

[issue16556] Update string.Formatter.vformat documentation to say "**kwargs"

2012-11-25 Thread Éric Araujo
Éric Araujo added the comment: Hello Taavi! It seems to me there is little value in changing all docs to use “**kwargs” instead of “**kwds”, given that this cannot cause actual problems. The interesting thing is that the function takes keyword args. For string.Formatter.vformat, I see no har

[issue16556] Update string.Formatter.vformat documentation to say "**kwargs"

2012-11-25 Thread Taavi Burns
New submission from Taavi Burns: The documentation says "**kwds" instead of "**kwargs", which is what the function signature uses. I noticed some other (internally-consistent) occurrences in string.rst. Is there any interest in replacing "**kwds" generally with "**kwargs"? It looks like 327 v