Author meador.inge
Recipients chris.jerdonek, docs@python, meador.inge, ncoghlan, terry.reedy
Date 2012-07-29.02:26:39
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1343528800.38.0.609944617048.issue15355@psf.upfronthosting.co.za>
In-reply-to
Content
Hmmm, in your original description you say that the 'generator.__next__' documentation should be changed, but in the patch the note applies to all generator methods.  From looking at the code it seems that the patch is correct and that '__next__', 'send', 'throw', and 'close' can all raise the "already executing" exception via 'gen_send_ex'.  Documenting this behavior seems reasonable, but you should probably mention what exception gets raises.

BTW, you don't need to make the Misc/NEWS changes a part of your patches.  A core dev will write that for you and since Misc/NEWS is changed so much it might conflict and make patches harder to apply across similar branches (say 3.2 and 3.3).
History
Date User Action Args
2012-07-29 02:26:40meador.ingesetrecipients: + meador.inge, terry.reedy, ncoghlan, chris.jerdonek, docs@python
2012-07-29 02:26:40meador.ingesetmessageid: <1343528800.38.0.609944617048.issue15355@psf.upfronthosting.co.za>
2012-07-29 02:26:39meador.ingelinkissue15355 messages
2012-07-29 02:26:39meador.ingecreate