From: Georg Brandl <georg@python.org>
Date: Sun, 1 Aug 2010 20:54:30 +0000 (+0000)
Subject: Recorded merge of revisions 83444 via svnmerge from
X-Git-Tag: v2.7.1rc1~513
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e27d044769166e47310345e7d538f54e1dac04d1;p=python

Recorded merge of revisions 83444 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/branches/py3k

........
  r83444 | georg.brandl | 2010-08-01 22:51:02 +0200 (So, 01 Aug 2010) | 1 line

  Revert r83395, it introduces test failures and is not necessary anyway since we now have to nul-terminate the string anyway.
........
---

diff --git a/Objects/unicodeobject.c b/Objects/unicodeobject.c
index 42e0c9fec6..f373eeb0ba 100644
--- a/Objects/unicodeobject.c
+++ b/Objects/unicodeobject.c
@@ -294,7 +294,7 @@ int unicode_resize(register PyUnicodeObject *unicode,
 }
 
 /* We allocate one more byte to make sure the string is
-   Ux0000 terminated -- XXX is this needed ?
+   Ux0000 terminated; some code relies on that.
 
    XXX This allocator could further be enhanced by assuring that the
    free list never reduces its size below 1.
@@ -3067,7 +3067,7 @@ PyObject *unicodeescape_string(const Py_UNICODE *s,
 
             ch2 = *s++;
             size--;
-            if (ch2 >= 0xDC00 && ch2 <= 0xDFFF && size) {
+            if (ch2 >= 0xDC00 && ch2 <= 0xDFFF) {
                 ucs = (((ch & 0x03FF) << 10) | (ch2 & 0x03FF)) + 0x00010000;
                 *p++ = '\\';
                 *p++ = 'U';
@@ -3316,7 +3316,7 @@ PyObject *PyUnicode_EncodeRawUnicodeEscape(const Py_UNICODE *s,
 
                 ch2 = *s++;
                 size--;
-                if (ch2 >= 0xDC00 && ch2 <= 0xDFFF && size) {
+                if (ch2 >= 0xDC00 && ch2 <= 0xDFFF) {
                     ucs = (((ch & 0x03FF) << 10) | (ch2 & 0x03FF)) + 0x00010000;
                     *p++ = '\\';
                     *p++ = 'U';