Message105497
This must be retargeted to 3.2. Also, the patch lacks some tests.
It seems PEP 8 compliance could be better: function names shouldn't be CamelCased.
Is LocalSynchronize() an implementation detail rather than a public API? If so, I think it would be better if it started with an underscore. |
|
| Date |
User |
Action |
Args |
| 2010-05-11 11:12:04 | pitrou | set | recipients:
+ pitrou, josiahcarlson, giampaolo.rodola |
| 2010-05-11 11:12:04 | pitrou | set | messageid: <[email protected]> |
| 2010-05-11 11:12:02 | pitrou | link | issue8684 messages |
| 2010-05-11 11:12:01 | pitrou | create | |
|