Message131925
Michael, in general your approach looks sound and is much easier to read and comprehend than the original code (which, as the comments say, was never refined from the original quick and dirty hack). However, rather than dynamically defining sub-functions each time body_encode is called, I've moved the body-construction logic almost completely out of body_encode into a helper object. I think this further clarifies the algorithm. I also used a simpler approach to end-of-list detection (enumerate). That change is more a matter of taste, but does have the advantage of taking fewer lines of code.
If you have time to review this and double check my changes (the tests pass, at least), that would be great. Otherwise I'll just go ahead and apply it. |
|
Date |
User |
Action |
Args |
2011-03-23 21:43:50 | r.david.murray | set | recipients:
+ r.david.murray, barry, michael.henry |
2011-03-23 21:43:49 | r.david.murray | set | messageid: <1300916629.78.0.673890352339.issue11606@psf.upfronthosting.co.za> |
2011-03-23 21:43:49 | r.david.murray | link | issue11606 messages |
2011-03-23 21:43:49 | r.david.murray | create | |
|