STINNER Victor added the comment:

Serhiy wrote on Rietveld:
"As far as this constant is repeated twice, it wastes memory and errorprone."

Oh, I expected marshal to be smart and use references, but it does not.

Here is a simpler patch which initialize all base85 tables at once using a 
"b85alphabet" variable which simplify also the initialization of the decoder 
table.

----------
Added file: http://bugs.python.org/file34421/base64_urlparse_lazy_init-3.patch

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

Reply via email to