Message144780
I attached a patch which replaces all asserts with checks that raise exceptions. I used my judgement in determining exception types but I might have been off in some places. Also, this patch replaces ALL asserts. It is possible that some of the internal functions should be kept using asserts but I am not familiar enough with the module to say which. I figured I'd better do the whole thing than reviewers can say where lines should be reverted to asserts. |
|
Date |
User |
Action |
Args |
2011-10-02 20:43:51 | vladris | set | recipients:
+ vladris, jnoller, BreamoreBoy |
2011-10-02 20:43:50 | vladris | set | messageid: <1317588230.22.0.452780658169.issue5001@psf.upfronthosting.co.za> |
2011-10-02 20:43:49 | vladris | link | issue5001 messages |
2011-10-02 20:43:49 | vladris | create | |
|