Alexandre Vassalotti <[EMAIL PROTECTED]> added the comment: On some benchmark of my own, I get a good 2x slowdown when this patch is applied. The benchmark consists of loading ~1.4 millions objects (mostly dict, list, str and int) from a pickle string. It is basically a torture test for the inner function calls overhead.
Anyway, the slowdown doesn't bother me much. I think a "stackless" pickle module would be well appreciated by many Python users. There is a few things that stops me from applying this patch. First, the patch will break subclasses of Pickler that relies on the save() method (Although the method is an implementation detail that shouldn't used). Therefore, any patches that modifies the API of save() is straight out for 2.x. And for Python 3.0, your patch should also remove recursion from the _pickle module (the transparent C accelerator for pickle). ---------- nosy: +alexandre.vassalotti _______________________________________ Python tracker <[EMAIL PROTECTED]> <http://bugs.python.org/issue3119> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
