Message112310
| Author |
bethard |
| Recipients |
bethard, georg.brandl, travistouchdown |
| Date |
2010年08月01日.10:50:22 |
| SpamBayes Score |
0.069184646 |
| Marked as misclassified |
No |
| Message-id |
<1280659824.57.0.60556997871.issue9399@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
The patch looks basically right. A few minor issues:
* "message=None," should probably be "message,", that is, message should not be allowed to default to None - I can't see any use case for this action without a message. I believe this means the body of __call__ can be simplified to::
self.file.write(self.message)
self.file.write("\n")
parser.exit()
* The other thing the patch needs is to update the test suite to add tests to make sure this behavior works. Take a look at test_argparse.py for how to do that.
* The last thing is that to have the greatest chance of having someone check this in, you'll want to make your patch against Python trunk as explained here:
http://www.python.org/dev/faq/#how-do-i-get-a-checkout-of-the-repository-read-only-or-read-write
http://www.python.org/dev/faq/#how-to-make-a-patch
Thanks! |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2010年08月01日 10:50:24 | bethard | set | recipients:
+ bethard, georg.brandl, travistouchdown |
| 2010年08月01日 10:50:24 | bethard | set | messageid: <1280659824.57.0.60556997871.issue9399@psf.upfronthosting.co.za> |
| 2010年08月01日 10:50:23 | bethard | link | issue9399 messages |
| 2010年08月01日 10:50:22 | bethard | create |
|