[Python-checkins] cpython: Backed out changeset b9f7b1bf36aa

[victor.stinner]

http://hg.python.org/cpython/rev/583a0321c7b9
changeset:   82503:583a0321c7b9
user:        Victor Stinner <victor.stinner at gmail.com>
date:        Wed Mar 06 01:09:24 2013 +0100
summary:
  Backed out changeset b9f7b1bf36aa

files:
  Misc/NEWS               |   3 ---
  Objects/unicodeobject.c |  19 +++++++------------
  2 files changed, 7 insertions(+), 15 deletions(-)


diff --git a/Misc/NEWS b/Misc/NEWS
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -10,9 +10,6 @@
 Core and Builtins
 -----------------
 
-- Issue #17223: Fix PyUnicode_FromUnicode() on Windows (16-bit wchar_t type)
-  to reject invalid UTF-16 surrogate.
-
 - Issue #17032: The "global" in the "NameError: global name 'x' is not defined"
   error message has been removed.  Patch by Ram Rachum.
 
diff --git a/Objects/unicodeobject.c b/Objects/unicodeobject.c
--- a/Objects/unicodeobject.c
+++ b/Objects/unicodeobject.c
@@ -1384,18 +1384,13 @@
 
     for (iter = begin; iter < end; ) {
 #if SIZEOF_WCHAR_T == 2
-        if (Py_UNICODE_IS_HIGH_SURROGATE(iter[0])) {
-            if ((iter+1) < end
-                && Py_UNICODE_IS_LOW_SURROGATE(iter[1]))
-            {
-                ch = Py_UNICODE_JOIN_SURROGATES(iter[0], iter[1]);
-                ++(*num_surrogates);
-                iter += 2;
-            }
-            else {
-                PyErr_SetString(PyExc_ValueError, "illegal UTF-16 surrogate");
-                return -1;
-            }
+        if (Py_UNICODE_IS_HIGH_SURROGATE(iter[0])
+            && (iter+1) < end
+            && Py_UNICODE_IS_LOW_SURROGATE(iter[1]))
+        {
+            ch = Py_UNICODE_JOIN_SURROGATES(iter[0], iter[1]);
+            ++(*num_surrogates);
+            iter += 2;
         }
         else
 #endif

-- 
Repository URL: http://hg.python.org/cpython