Author r.david.murray
Recipients ezio.melotti, lemburg, r.david.murray, rhansen
Date 2010-01-07.13:02:06
SpamBayes Score 6.61757e-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:08r.david.murraysetrecipients: + r.david.murray, lemburg, ezio.melotti, rhansen
2010-01-07 13:02:08r.david.murraysetmessageid: <1262869328.41.0.728063629472.issue7615@psf.upfronthosting.co.za>
2010-01-07 13:02:06r.david.murraylinkissue7615 messages
2010-01-07 13:02:06r.david.murraycreate