Marc-Andre Lemburg added the comment: On 14.11.2016 13:03, STINNER Victor wrote: > > STINNER Victor added the comment: > > iso6937.py: > >> # from utf-8 to iso6937 >> def iso6937_encode(input,errors,encoding_map): > > Wait, is this code for Python 3? Decode from UTF-8 and encode to ISO-6937 in > the same function seems strange to me.
The patch shows the file as UTF-8. In reality, it is decoding from Unicode strings. > I expected that the codec only implements two functions: encode text > (unicode) to ISO-6937 (bytes), decode bytes from ISO-6937 to text. > > Since the encoding is non trivial (multibyte), if we decide to add it, I > suggest to require unit tests. I would like to see unit tests on multibyte > strings, to check how the error handler is handled. +1 > In general, I would prefer to not embed too many codecs in Python, it has a > little cost to maintain these codecs. > > My rule is more to only added encodings used (in practice) as locale > encodings. This encoding is used in EPG data of various DVB television formats. As such it is in active use (even though it is very old). I think "active use" is a better approach to restricting ourselves to only locale encodings, since the latter are slowly converging towards UTF-8 :-) BTW: Once a charmap style codec is written, there is little change, so the maintenance is minimal. Codecs which include more active logic such as this one are different, of course, and therefore may potentially add more maintenance burden. ---------- _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue24339> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
