Message121845
| Author |
bethard |
| Recipients |
bethard, eric.araujo, ezio.melotti, maker, r.david.murray |
| Date |
2010年11月21日.03:06:45 |
| SpamBayes Score |
2.2235765e-06 |
| Marked as misclassified |
No |
| Message-id |
<1290308807.19.0.794584626411.issue10424@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
Yeah a new test class is fine.
And I checked the patch and it looks okay to me. My first thought was also "wait does that really work?" but I see that positionals are all marked as required when appropriate (look for the comment starting with "mark positional arguments as required").
I don't have time to test the patch right now, but if someone else does, I'm fine with this after the test for the new behavior is added. |
|