[Python-Dev] [Python-checkins] cpython (3.3): Issue #16641: Fix default values of sched.scheduler.enter arguments were
g.brandl at gmx.net
Sat Dec 29 23:17:03 CET 2012
On 12/29/2012 10:44 PM, Eli Bendersky wrote:
> > Executing the event means executing ``action(*argument, **kwargs)``.
> > - *argument* must be a sequence holding the parameters for *action*.
> > - *kwargs* must be a dictionary holding the keyword parameters for
> > + Optional *argument* argument must be a sequence holding the parameters
> > + for *action* if any used.
> > + Optional *kwargs* argument must be a dictionary holding the keyword
> > + parameters for *action* if any used.
> > I don't see how this change improves the documentation. To keep the grammar
> > correct and just state that the arguments are optional, I would simply replace
> > "must be" by "is". For example:
> > *argument* is a sequence holding the parameters for *action*.
> > This is short, and since the function signature clearly shows that
> argument has
> > a default value, I think it conveys the meaning it should.
> Hi Eli,
> I'm sure we non-native speakers are fine with any improvements you can make
> during commit review.
> I also wrote a private email to Serhiy proposing to help, but since you brought
> this up here: I think that my comment was constructive. What should have I done
Nothing at all. In case you read my comment as sarcasm, please don't. It
probably was a little unreflected.
> Go ahead and modify the phrasing in a separate commit? I see a couple of problems with that:
> 1. It can be somewhat disrespectful to a new committer, and I wanted to reach a
> consensus first.
I would not feel unrespected by someone correcting nits in my English grammar.
(Maybe it would be different for Python grammar :)
> 2. Serhiy diligently committed this into 3 or 4 different Python branches. With
> all due respect, going through the merge/push dance is far above the effort I'm
> willing to invest in this.
I understand. I probably would have not done it instantly either, but put it
on my todo list and committed when I had some other change as well.
> P.S. I would argue that you are more native-speaker than myself w.r.t. English :-)
Then I was mistaken there. (Are there multiple levels of nativeness? :) Or is
it because English is half-Germanic?)
More information about the Python-Dev