Message121557
Here is the patch in the current state which includes the changes in issue2001_c.diff as well as most of the changes Éric suggested.
Still to do:
* Use the with statement in several places to ensure closing.
* Add tests for the server.
I did try to make the header a bit less cluttered, but I'm not completely happy with it yet, but I think it will be good enough for now. Fixing the HTML probably should be a separate issue where we can add a style sheet at the same time. Lets get this patch in first.
This is also reitveld:
http://codereview.appspot.com/3151042/ |
|
Date |
User |
Action |
Args |
2010-11-19 17:59:52 | ron_adam | set | recipients:
+ ron_adam, ping, georg.brandl, ncoghlan, belopolsky, eric.araujo, r.liebscher |
2010-11-19 17:59:51 | ron_adam | set | messageid: <1290189591.52.0.283323670262.issue2001@psf.upfronthosting.co.za> |
2010-11-19 17:59:49 | ron_adam | link | issue2001 messages |
2010-11-19 17:59:49 | ron_adam | create | |
|