Message107569
I am attaching a new patch, issue5094e.diff which addresses most of Mark's comments. I left out repr() because two opinions were voiced on IRC with respect to datetime. prefix. I would like to give it some more thought even though I am leaning towards compatibility with existing reprs.
I did not make td argument optional and did not allow timezone o be subclassed because these seem to be mutually exclusive options. (If td is optional in base class, it must be optional in subclasses per Liskov's principle severely limiting utility of subclasses.) Let's address this separately. |
|
Date |
User |
Action |
Args |
2010-06-11 18:48:51 | belopolsky | set | recipients:
+ belopolsky, tim.peters, doerwalter, brett.cannon, mark.dickinson, ggenellina, pitrou, techtonik, ajaksu2, kawai, eric.araujo, r.david.murray, rafe, daniel.urban, l0nwlf |
2010-06-11 18:48:50 | belopolsky | set | messageid: <1276282130.79.0.539298542865.issue5094@psf.upfronthosting.co.za> |
2010-06-11 18:48:48 | belopolsky | link | issue5094 messages |
2010-06-11 18:48:48 | belopolsky | create | |
|