Message97489
The patch looks good to me, except that one test fails. You seem to have inadvertently deleted the '=' from the safe chars list in the test.
We should also add a test for the '' case. And presumably the docs requested in the other ticket... |
|
Date |
User |
Action |
Args |
2010-01-10 01:58:08 | r.david.murray | set | recipients:
+ r.david.murray, tim.peters, georg.brandl, eric.smith, jjwiseman, bgertzfield |
2010-01-10 01:58:07 | r.david.murray | set | messageid: <1263088687.55.0.960723576615.issue7507@psf.upfronthosting.co.za> |
2010-01-10 01:58:05 | r.david.murray | link | issue7507 messages |
2010-01-10 01:58:05 | r.david.murray | create | |
|