[issue19619] Blacklist base64, hex, ... codecs from bytes.decode() and str.encode()
Marc-Andre Lemburg
report at bugs.python.org
Fri Nov 22 12:24:16 CET 2013
Marc-Andre Lemburg added the comment:
Nick: I've had a look at your second patch. A couple of notes:
* I think you should add the flag to the constructor of the CodecInfo
tuple subclass and then set this in the resp. codecs. The extra
constructor class method looks too much like a hack and is
not needed.
* The comment in codecs.h should read:
"""
Checks the encoding against a list of codecs which do not
implement a str<->bytes encoding before attempting the
operation.
Please note that these APIs are internal and should not
be used in Python C extensions.
"""
Regarding Victor's suggestion to use a separate registry dict
for this: I'm definitely -1 on this.
The black listing is a very special case only used for the
.encode()/.decode() methods and otherwise doesn't have anything to do
with the codecs sub-system. It doesn't make sense to change the design
of the registry just to implement this one special case.
----------
_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue19619>
_______________________________________
More information about the Python-bugs-list
mailing list