Ezio Melotti added the comment:

The fix looks ok, however the test seem to contain unnecessary code and it 
duplicates most of the previous test.
I left a more detailed review on rietveld (see the "review" link next to the 
patch).

----------
type:  -> behavior

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue25320>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to