[Python-Dev] issue 2170 review
Brett Cannon
brett at python.org
Mon Mar 23 03:21:26 CET 2009
On Sun, Mar 22, 2009 at 18:49, R. David Murray <rdmurray at bitdance.com>wrote:
> Instead of looking at a bunch of issues last week the way I'd originally
> intended, I wound up doing a review of a particular patch submission,
> issue 2170. This is a refactoring of the 'normalize' method of
> xml.dom.minidom. I wound up redoing the patch with a different
> refactoring after finding out that the submitted patch was slower than
> the original code. The revised patch is faster than the original code
> (completes in 1/2 the time of the original code when run on a decent
> sized document that needed no normalization done).
>
> I think the issue stage should be set to 'patch review', as in, it is ready
> for developer review. The author of the original patch has reviewed my
> revised patch and concurs. The patch includes additional tests.
>
I have set the stage.
>
> Now that I've learned more about how the tracker is organized, and read
> the most-helpful http://www.python.org/dev/workflow/ document, I would
> be happy to go through and triage issues for which that hasn't yet been
> done, if I can be given permission to do so.
Done!
-Brett
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.python.org/pipermail/python-dev/attachments/20090322/3801593e/attachment.htm>
More information about the Python-Dev
mailing list