Re: [Python-Dev] [Python-checkins] cpython: Revert the Lib/test/test_bigmem.py changes from commit 17891566a478 (and a

On 25 March 2011 17:09, Antoine Pitrou wrote:

On Fri, 25 Mar 2011 17:44:26 +0100 Éric Araujo wrote:

...

Hi,

...

changeset: 68921:11dc3f270594 user: Thomas Wouters date: Fri Mar 25 11:42:37 2011 +0100 summary: Revert the Lib/test/test_bigmem.py changes from commit 17891566a478 (and a few other assertEqual tests that snuck in), and expand the docstrings and comments explaining why and how these tests are supposed to work.

Your commit message does not explain why you reverted the changes. The specific assert* methods give more useful messages than assertEqual in case of failure.

Because they don't go well with huge inputs?

...

...

...

s = "x" * (2**29) case.assertEqual(s + "a", s + "b") Traceback (most recent call last): File "<stdin>", line 1, in <module> File "/home/antoine/cpython/default/Lib/unittest/case.py", line 643, in assertEqual assertion_func(first, second, msg=msg) File "/home/antoine/cpython/default/Lib/unittest/case.py", line 984, in assertMultiLineEqual secondlines = [second + '\n'] MemoryError

http://bugs.python.org/issue11763 Michael

(of course, the functions could just be fixed)

_______________________________________________ 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/fuzzyman%40voidspace.org.u...

-- http://www.voidspace.org.uk/ May you do good and not evil May you find forgiveness for yourself and forgive others May you share freely, never taking more than you give. -- the sqlite blessing http://www.sqlite.org/different.html