Author pitrou
Recipients akuchling, amaury.forgeotdarc, ezio.melotti, georg.brandl, jimjjewett, mark, moreati, mrabarnett, nneonneo, pitrou, rsc, timehorse
Date 2009-02-09.19:09:54
SpamBayes Score 2.01952e-06
Marked as misclassified No
Message-id <1234206597.21.0.973712300136.issue2636@psf.upfronthosting.co.za>
In-reply-to
Content
Besides the fact that this is probably great work, I really wonder who
will have enough time and skills to review such a huge patch... :-S

In any case, some recommendations:
- please provide patches against trunk; there is no way such big changes
will get committed against 2.6, which is in maintenance mode
- avoid, as far as possible, doing changes in style, whitespace or
indentation; this will make the patch slightly smaller or cleaner
- avoid C++-style comments (use /* ... */ instead)
- don't hesitate to add extensive comments and documentation about what
you've added

Once you think your patch is ready, you may post it to
http://codereview.appspot.com/, in the hope that it makes reviewing easier.
History
Date User Action Args
2009-02-09 19:09:57pitrousetrecipients: + pitrou, akuchling, georg.brandl, jimjjewett, amaury.forgeotdarc, nneonneo, rsc, timehorse, mark, ezio.melotti, mrabarnett, moreati
2009-02-09 19:09:57pitrousetmessageid: <1234206597.21.0.973712300136.issue2636@psf.upfronthosting.co.za>
2009-02-09 19:09:55pitroulinkissue2636 messages
2009-02-09 19:09:54pitroucreate