diff options
author | Ezio Melotti <ezio.melotti@gmail.com> | 2011-04-15 16:52:35 +0300 |
---|---|---|
committer | Ezio Melotti <ezio.melotti@gmail.com> | 2011-04-15 16:52:35 +0300 |
commit | 73adc447d3418d63c707dce17c81fcdcc91fdb18 (patch) | |
tree | 95b502f6a26a25c74a0c50b2a395742da250b934 /Python/peephole.c | |
parent | 4166763366116cec01f9f704799c3b74ac4fbd8c (diff) | |
parent | 3913ad265314731c5a8c42b30ca7b7699db94b23 (diff) | |
download | cpython-73adc447d3418d63c707dce17c81fcdcc91fdb18.tar.gz |
#5057: Merge with 3.2.
Diffstat (limited to 'Python/peephole.c')
-rw-r--r-- | Python/peephole.c | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/Python/peephole.c b/Python/peephole.c index 69f61617f8..781498ecf2 100644 --- a/Python/peephole.c +++ b/Python/peephole.c @@ -183,6 +183,24 @@ fold_binops_on_constants(unsigned char *codestr, PyObject *consts, PyObject **ob break; case BINARY_SUBSCR: newconst = PyObject_GetItem(v, w); + /* #5057: if v is unicode, there might be differences between + wide and narrow builds in cases like '\U00012345'[0]. + Wide builds will return a non-BMP char, whereas narrow builds + will return a surrogate. In both the cases skip the + optimization in order to produce compatible pycs. + */ + if (newconst != NULL && + PyUnicode_Check(v) && PyUnicode_Check(newconst)) { + Py_UNICODE ch = PyUnicode_AS_UNICODE(newconst)[0]; +#ifdef Py_UNICODE_WIDE + if (ch > 0xFFFF) { +#else + if (ch >= 0xD800 && ch <= 0xDFFF) { +#endif + Py_DECREF(newconst); + return 0; + } + } break; case BINARY_LSHIFT: newconst = PyNumber_Lshift(v, w); |