Marc-Andre Lemburg <mal@lemburg.com>:
Added code so that .isXXX() testing returns 0 for emtpy strings.
diff --git a/Objects/stringobject.c b/Objects/stringobject.c
index 5d754b0..a254019 100644
--- a/Objects/stringobject.c
+++ b/Objects/stringobject.c
@@ -1927,6 +1927,10 @@
isspace(*p))
return PyInt_FromLong(1);
+ /* Special case for empty strings */
+ if (PyString_GET_SIZE(self) == 0)
+ return PyInt_FromLong(0);
+
e = p + PyString_GET_SIZE(self);
for (; p < e; p++) {
if (!isspace(*p))
@@ -1956,6 +1960,10 @@
isdigit(*p))
return PyInt_FromLong(1);
+ /* Special case for empty strings */
+ if (PyString_GET_SIZE(self) == 0)
+ return PyInt_FromLong(0);
+
e = p + PyString_GET_SIZE(self);
for (; p < e; p++) {
if (!isdigit(*p))
@@ -1985,6 +1993,10 @@
if (PyString_GET_SIZE(self) == 1)
return PyInt_FromLong(islower(*p) != 0);
+ /* Special case for empty strings */
+ if (PyString_GET_SIZE(self) == 0)
+ return PyInt_FromLong(0);
+
e = p + PyString_GET_SIZE(self);
cased = 0;
for (; p < e; p++) {
@@ -2017,6 +2029,10 @@
if (PyString_GET_SIZE(self) == 1)
return PyInt_FromLong(isupper(*p) != 0);
+ /* Special case for empty strings */
+ if (PyString_GET_SIZE(self) == 0)
+ return PyInt_FromLong(0);
+
e = p + PyString_GET_SIZE(self);
cased = 0;
for (; p < e; p++) {
@@ -2050,6 +2066,10 @@
if (PyString_GET_SIZE(self) == 1)
return PyInt_FromLong(isupper(*p) != 0);
+ /* Special case for empty strings */
+ if (PyString_GET_SIZE(self) == 0)
+ return PyInt_FromLong(0);
+
e = p + PyString_GET_SIZE(self);
cased = 0;
previous_is_cased = 0;
diff --git a/Objects/unicodeobject.c b/Objects/unicodeobject.c
index eaa381b..bfc59dd 100644
--- a/Objects/unicodeobject.c
+++ b/Objects/unicodeobject.c
@@ -3306,6 +3306,10 @@
if (PyUnicode_GET_SIZE(self) == 1)
return PyInt_FromLong(Py_UNICODE_ISLOWER(*p) != 0);
+ /* Special case for empty strings */
+ if (PyString_GET_SIZE(self) == 0)
+ return PyInt_FromLong(0);
+
e = p + PyUnicode_GET_SIZE(self);
cased = 0;
for (; p < e; p++) {
@@ -3339,6 +3343,10 @@
if (PyUnicode_GET_SIZE(self) == 1)
return PyInt_FromLong(Py_UNICODE_ISUPPER(*p) != 0);
+ /* Special case for empty strings */
+ if (PyString_GET_SIZE(self) == 0)
+ return PyInt_FromLong(0);
+
e = p + PyUnicode_GET_SIZE(self);
cased = 0;
for (; p < e; p++) {
@@ -3374,6 +3382,10 @@
return PyInt_FromLong((Py_UNICODE_ISTITLE(*p) != 0) ||
(Py_UNICODE_ISUPPER(*p) != 0));
+ /* Special case for empty strings */
+ if (PyString_GET_SIZE(self) == 0)
+ return PyInt_FromLong(0);
+
e = p + PyUnicode_GET_SIZE(self);
cased = 0;
previous_is_cased = 0;
@@ -3418,6 +3430,10 @@
Py_UNICODE_ISSPACE(*p))
return PyInt_FromLong(1);
+ /* Special case for empty strings */
+ if (PyString_GET_SIZE(self) == 0)
+ return PyInt_FromLong(0);
+
e = p + PyUnicode_GET_SIZE(self);
for (; p < e; p++) {
if (!Py_UNICODE_ISSPACE(*p))
@@ -3446,6 +3462,10 @@
Py_UNICODE_ISDECIMAL(*p))
return PyInt_FromLong(1);
+ /* Special case for empty strings */
+ if (PyString_GET_SIZE(self) == 0)
+ return PyInt_FromLong(0);
+
e = p + PyUnicode_GET_SIZE(self);
for (; p < e; p++) {
if (!Py_UNICODE_ISDECIMAL(*p))
@@ -3474,6 +3494,10 @@
Py_UNICODE_ISDIGIT(*p))
return PyInt_FromLong(1);
+ /* Special case for empty strings */
+ if (PyString_GET_SIZE(self) == 0)
+ return PyInt_FromLong(0);
+
e = p + PyUnicode_GET_SIZE(self);
for (; p < e; p++) {
if (!Py_UNICODE_ISDIGIT(*p))
@@ -3502,6 +3526,10 @@
Py_UNICODE_ISNUMERIC(*p))
return PyInt_FromLong(1);
+ /* Special case for empty strings */
+ if (PyString_GET_SIZE(self) == 0)
+ return PyInt_FromLong(0);
+
e = p + PyUnicode_GET_SIZE(self);
for (; p < e; p++) {
if (!Py_UNICODE_ISNUMERIC(*p))