Commit 54d951ce authored by Guido van Rossum's avatar Guido van Rossum

Fix the tests for various anomalies in the string-to-numbers

conversions.  Formerly, for example, int('-') would return 0 instead
of raising ValueError, and int(' 0') would raise ValueError
(complaining about a null byte!) instead of 0...
parent 4a582494
......@@ -67,12 +67,12 @@ int_from_string(v)
s = PyString_AS_STRING(v);
while (*s && isspace(Py_CHARMASK(*s)))
s++;
if (s[0] == '\0') {
PyErr_SetString(PyExc_ValueError, "empty string for int()");
return NULL;
}
errno = 0;
x = PyOS_strtol(s, &end, 10);
if (end == s || !isdigit(end[-1])) {
PyErr_SetString(PyExc_ValueError, "no digits in int constant");
return NULL;
}
while (*end && isspace(Py_CHARMASK(*end)))
end++;
if (*end != '\0') {
......@@ -80,7 +80,7 @@ int_from_string(v)
PyErr_SetString(PyExc_ValueError, buffer);
return NULL;
}
else if (end-s != PyString_GET_SIZE(v)) {
else if (end != PyString_AS_STRING(v) + PyString_GET_SIZE(v)) {
PyErr_SetString(PyExc_ValueError,
"null byte in argument for int()");
return NULL;
......@@ -104,10 +104,6 @@ long_from_string(v)
s = PyString_AS_STRING(v);
while (*s && isspace(Py_CHARMASK(*s)))
s++;
if (s[0] == '\0') {
PyErr_SetString(PyExc_ValueError, "empty string for long()");
return NULL;
}
x = PyLong_FromString(s, &end, 10);
if (x == NULL)
return NULL;
......@@ -119,9 +115,9 @@ long_from_string(v)
Py_DECREF(x);
return NULL;
}
else if (end-s != PyString_GET_SIZE(v)) {
else if (end != PyString_AS_STRING(v) + PyString_GET_SIZE(v)) {
PyErr_SetString(PyExc_ValueError,
"null byte in argument for float()");
"null byte in argument for long()");
return NULL;
}
return x;
......@@ -154,7 +150,7 @@ float_from_string(v)
PyErr_SetString(PyExc_ValueError, buffer);
return NULL;
}
else if (end-s != PyString_GET_SIZE(v)) {
else if (end != PyString_AS_STRING(v) + PyString_GET_SIZE(v)) {
PyErr_SetString(PyExc_ValueError,
"null byte in argument for float()");
return NULL;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment