Re: [Python-Dev] issue 2170 review

2009-03-22 Thread Brett Cannon
On Sun, Mar 22, 2009 at 18:49, R. David Murray 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
___
Python-Dev mailing list
Python-Dev@python.org
http://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: 
http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com


[Python-Dev] issue 2170 review

2009-03-22 Thread R. David Murray

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.

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.

--
R. David Murray   http://www.bitdance.com
___
Python-Dev mailing list
Python-Dev@python.org
http://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: 
http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com