Message164707
Thanks for the patch. It looks good to me, but I think that adding support for Ellipsis and adding a new parameter to allow extra names are two different requests, but someone more knowledgeable than me about AST may judge differently. My rationale is that supporting Ellipsis literals is a no-brainer addition, whereas adding an argument to support things that are not actual Python literals may be rejected. You don’t have to split your patch in two anyway, it’s easy to do if the dev who will commit wants to.
Each change would also need a doc update. |
|
Date |
User |
Action |
Args |
2012-07-06 00:21:06 | eric.araujo | set | recipients:
+ eric.araujo, georg.brandl, amaury.forgeotdarc, mark.dickinson, benjamin.peterson, daniel.urban, JBernardo |
2012-07-06 00:21:05 | eric.araujo | set | messageid: <1341534065.7.0.124598202114.issue15245@psf.upfronthosting.co.za> |
2012-07-06 00:21:05 | eric.araujo | link | issue15245 messages |
2012-07-06 00:21:04 | eric.araujo | create | |
|