import cPickle

res=[]
for x in range(1,2000):
    res.append(dict(doc=x, similar=[]))

cPickle.dumps(res)


Traceback (most recent call last):
  File "pi.py", line 10, in <module>
cPickle.dumps(res)
RuntimeError: maximum recursion depth exceeded

svn r64471 seems to cause the problematic behaviour.

facundo, you committed that one.

This example works before my patch, exactly!

The reason of the patch I applied is that in some cases a Recursion
error should be raised, but it didn't happen, causing some serious
issues later.

I'm putting in copy to cuerty, for him to analyze if this case should
have caused Recursion error in the first place, or not.

In any case, it's a good example. Ralf, would you mind to generate a
test case from this example?

Thank you all!

Apparently there are some self->nesting-- calls missing in batch_list
and batch_dict.

Of course it should not raise an RecursionError.

for reference:
http://bugs.python.org/issue2702 is the original bugreport.

here is a test case. I cannot run it however:
~/pydev/trunk/ ./python Lib/test/test_cpickle.py                   
ralf@red ok
Traceback (most recent call last):
  File "Lib/test/test_cpickle.py", line 3, in <module>
from test.pickletester import AbstractPickleTests, AbstractPickleModuleTests
ImportError: No module named pickletester

???

the test works as expected (i.e. it fails). The problem I had was that 
some Bittorrent bencode also installed a test package for me.


======================================================================
ERROR: test_flat_list (__main__.cPickleFlatList)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "Lib/test/test_cpickle.py", line 113, in test_flat_list
    cPickle.dumps(lst)
RuntimeError: maximum recursion depth exceeded

----------------------------------------------------------------------
Ran 159 tests in 0.457s

FAILED (errors=1)
Traceback (most recent call last):
  File "Lib/test/test_cpickle.py", line 128, in <module>
test_main()
  File "Lib/test/test_cpickle.py", line 124, in test_main
cPickleFlatList,
  File "/home/ralf/pydev/trunk/Lib/test/test_support.py", line 714, in
run_unittest
_run_suite(suite)
  File "/home/ralf/pydev/trunk/Lib/test/test_support.py", line 697, in
_run_suite
raise TestFailed(err)
test.test_support.TestFailed: Traceback (most recent call last):
  File "Lib/test/test_cpickle.py", line 113, in test_flat_list
    cPickle.dumps(lst)
RuntimeError: maximum recursion depth exceeded

btw. this should be a release blocker.

I reverted the patch, commented out the previous test, and included this
one in the test suite, to never break it again, ;)

I hope we now find a solution to the issue #2702.

Thank you!!