Message97352
| Author |
r.david.murray |
| Recipients |
ezio.melotti, lemburg, r.david.murray, rhansen |
| Date |
2010年01月07日.13:02:06 |
| SpamBayes Score |
6.6175704e-07 |
| Marked as misclassified |
No |
| Message-id |
<1262869328.41.0.728063629472.issue7615@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
Does the last patch obsolete the first two? If so please delete the obsolete ones.
I imagine there are might be small doc updates required, as well.
I haven't looked at the patch itself, but concerning your test patch: your try/except style is unnecessary, I think. Better to just let the syntax error bubble up on its own. After all, you don't *know* that the SyntaxError is because the quotes aren't escaped. I've run into other unit tests in the test suite where the author made such an assumption, which turned out to be false and confused me for a bit while debugging the failure. I had to remove the code producing the mistaken reason message in order to see the real problem. So it's better to just let the real error show up in the first place, in my experience. |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2010年01月07日 13:02:08 | r.david.murray | set | recipients:
+ r.david.murray, lemburg, ezio.melotti, rhansen |
| 2010年01月07日 13:02:08 | r.david.murray | set | messageid: <1262869328.41.0.728063629472.issue7615@psf.upfronthosting.co.za> |
| 2010年01月07日 13:02:06 | r.david.murray | link | issue7615 messages |
| 2010年01月07日 13:02:06 | r.david.murray | create |
|