summaryrefslogtreecommitdiffstats
path: root/Python
diff options
context:
space:
mode:
authorEzio Melotti <ezio.melotti@gmail.com>2011-04-15 13:52:35 (GMT)
committerEzio Melotti <ezio.melotti@gmail.com>2011-04-15 13:52:35 (GMT)
commite1adeeb7fd25acb4649d0eae314c5b9dc02348d4 (patch)
treeea15ce1195e23707d74721f12fc5a8f9889996e4 /Python
parentb496159603e5cc31904a894166ecce52fea01a1c (diff)
parent71e84a6e27da7bfa8351d03ba8e52eb0b5150fb2 (diff)
downloadcpython-e1adeeb7fd25acb4649d0eae314c5b9dc02348d4.zip
cpython-e1adeeb7fd25acb4649d0eae314c5b9dc02348d4.tar.gz
cpython-e1adeeb7fd25acb4649d0eae314c5b9dc02348d4.tar.bz2
#5057: Merge with 3.2.
Diffstat (limited to 'Python')
-rw-r--r--Python/peephole.c18
1 files changed, 18 insertions, 0 deletions
diff --git a/Python/peephole.c b/Python/peephole.c
index 69f6161..781498e 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);