diff options
author | Victor Stinner <victor.stinner@gmail.com> | 2016-03-01 20:30:30 (GMT) |
---|---|---|
committer | Victor Stinner <victor.stinner@gmail.com> | 2016-03-01 20:30:30 (GMT) |
commit | 6c9aa8f2bf46e28de74ce11d85bcd448c0d0d529 (patch) | |
tree | d60ac5d1a1c8210af5403fd948a68f802b8ba35f | |
parent | bb0dbd583bdac8f1bfb9f0a45e7014b2663d5729 (diff) | |
download | cpython-6c9aa8f2bf46e28de74ce11d85bcd448c0d0d529.zip cpython-6c9aa8f2bf46e28de74ce11d85bcd448c0d0d529.tar.gz cpython-6c9aa8f2bf46e28de74ce11d85bcd448c0d0d529.tar.bz2 |
Fix str.translate()
Issue #26464: Fix str.translate() when string is ASCII and first replacements
removes character, but next replacement uses a non-ASCII character or a string
longer than 1 character. Regression introduced in Python 3.5.0.
-rw-r--r-- | Lib/test/test_unicode.py | 4 | ||||
-rw-r--r-- | Misc/NEWS | 4 | ||||
-rw-r--r-- | Objects/unicodeobject.c | 7 |
3 files changed, 12 insertions, 3 deletions
diff --git a/Lib/test/test_unicode.py b/Lib/test/test_unicode.py index fac8b7b..b258db1 100644 --- a/Lib/test/test_unicode.py +++ b/Lib/test/test_unicode.py @@ -347,6 +347,10 @@ class UnicodeTest(string_tests.CommonTest, "[a]") self.assertEqual("[\xe9]".translate(str.maketrans({'\xe9': None})), "[]") + self.assertEqual('axb'.translate(str.maketrans({'a': None, 'b': '123'})), + "x123") + self.assertEqual('axb'.translate(str.maketrans({'a': None, 'b': '\xe9'})), + "x\xe9") # invalid Unicode characters invalid_char = 0x10ffff+1 @@ -10,6 +10,10 @@ Release date: tba Core and Builtins ----------------- +- Issue #26464: Fix str.translate() when string is ASCII and first replacements + removes character, but next replacement uses a non-ASCII character or a + string longer than 1 character. Regression introduced in Python 3.5.0. + - Issue #22836: Ensure exception reports from PyErr_Display() and PyErr_WriteUnraisable() are sensible even when formatting them produces secondary errors. This affects the reports produced by diff --git a/Objects/unicodeobject.c b/Objects/unicodeobject.c index d4c9aae..a1ee776 100644 --- a/Objects/unicodeobject.c +++ b/Objects/unicodeobject.c @@ -8574,7 +8574,8 @@ unicode_fast_translate_lookup(PyObject *mapping, Py_UCS1 ch, translated into writer, raise an exception and return -1 on error. */ static int unicode_fast_translate(PyObject *input, PyObject *mapping, - _PyUnicodeWriter *writer, int ignore) + _PyUnicodeWriter *writer, int ignore, + Py_ssize_t *input_pos) { Py_UCS1 ascii_table[128], ch, ch2; Py_ssize_t len; @@ -8621,6 +8622,7 @@ unicode_fast_translate(PyObject *input, PyObject *mapping, exit: writer->pos = out - PyUnicode_1BYTE_DATA(writer->buffer); + *input_pos = in - PyUnicode_1BYTE_DATA(input); return res; } @@ -8666,7 +8668,7 @@ _PyUnicode_TranslateCharmap(PyObject *input, ignore = (errors != NULL && strcmp(errors, "ignore") == 0); - res = unicode_fast_translate(input, mapping, &writer, ignore); + res = unicode_fast_translate(input, mapping, &writer, ignore, &i); if (res < 0) { _PyUnicodeWriter_Dealloc(&writer); return NULL; @@ -8674,7 +8676,6 @@ _PyUnicode_TranslateCharmap(PyObject *input, if (res == 1) return _PyUnicodeWriter_Finish(&writer); - i = writer.pos; while (i<size) { /* try to encode it */ int translate; |