Message124516
Why not simply remove the additional check in line 168 and leave the responsibility to check the validity of its input to the unescape function (be it explicitly or, like now, lazily). That way, the code changes are minimal, the existing test covers the current issue, and the function gets more robust.
By the way, I came across this function via Stackoverflow:
http://stackoverflow.com/questions/2087370 |
|
Date |
User |
Action |
Args |
2010-12-22 19:23:19 | Martin.Potthast | set | recipients:
+ Martin.Potthast, orsenthil, r.david.murray |
2010-12-22 19:23:19 | Martin.Potthast | set | messageid: <1293045799.8.0.720226229295.issue10759@psf.upfronthosting.co.za> |
2010-12-22 19:23:16 | Martin.Potthast | link | issue10759 messages |
2010-12-22 19:23:16 | Martin.Potthast | create | |
|