Message263216
| Author |
martin.panter |
| Recipients |
Christophe.Guillon, abacabadabacaba, amcnabb, andersk, bethard, cben, danielsh, davidben, drm, eric.araujo, eric.smith, gdb, gfxmonk, martin.panter, memeplex, nelhage, paul.j3, r.david.murray, skilletaudio |
| Date |
2016年04月12日.01:24:59 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1460424300.11.0.701386505976.issue9334@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
My main concern with the patch is that it only half fixes the problem. It sounds like it will allow parsing "--opt -x" (if "-x" is not registered as an option), but will still refuse "--opt -h", assuming "-h" is registered by default. What is the barrier to parsing an argument to the option syntax independently of what option names are registered?
Also the name "args_default_to_positional=True" name is both unwieldy and vague to me. The purpose seems to be to disable option-lookalike-strings from being reserved. Maybe call it something like "reserve_all_options=False" or "reserve_unregistered_options=False"?
I left some thoughts in the code review for the documentation too. |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2016年04月12日 01:25:00 | martin.panter | set | recipients:
+ martin.panter, cben, amcnabb, bethard, eric.smith, eric.araujo, r.david.murray, memeplex, gfxmonk, andersk, abacabadabacaba, gdb, nelhage, drm, davidben, paul.j3, skilletaudio, Christophe.Guillon, danielsh |
| 2016年04月12日 01:25:00 | martin.panter | set | messageid: <1460424300.11.0.701386505976.issue9334@psf.upfronthosting.co.za> |
| 2016年04月12日 01:25:00 | martin.panter | link | issue9334 messages |
| 2016年04月12日 01:24:59 | martin.panter | create |
|