Commit 8f7e09ed authored by Anthony Baxter's avatar Anthony Baxter

C++ compiler changes. casts, rename variables with reserved names.

parent 056a9de9
...@@ -308,7 +308,7 @@ unpack_string(LogReaderObject *self, PyObject **pvalue) ...@@ -308,7 +308,7 @@ unpack_string(LogReaderObject *self, PyObject **pvalue)
if ((err = unpack_packed_int(self, &len, 0))) if ((err = unpack_packed_int(self, &len, 0)))
return err; return err;
buf = malloc(len); buf = (char *)malloc(len);
for (i=0; i < len; i++) { for (i=0; i < len; i++) {
ch = fgetc(self->logfp); ch = fgetc(self->logfp);
buf[i] = ch; buf[i] = ch;
...@@ -1403,7 +1403,7 @@ get_version_string(void) ...@@ -1403,7 +1403,7 @@ get_version_string(void)
++rev; ++rev;
while (rev[i] != ' ' && rev[i] != '\0') while (rev[i] != ' ' && rev[i] != '\0')
++i; ++i;
buffer = malloc(i + 1); buffer = (char *)malloc(i + 1);
if (buffer != NULL) { if (buffer != NULL) {
memmove(buffer, rev, i); memmove(buffer, rev, i);
buffer[i] = '\0'; buffer[i] = '\0';
......
...@@ -446,16 +446,16 @@ strop_lower(PyObject *self, PyObject *args) ...@@ -446,16 +446,16 @@ strop_lower(PyObject *self, PyObject *args)
{ {
char *s, *s_new; char *s, *s_new;
Py_ssize_t i, n; Py_ssize_t i, n;
PyObject *new; PyObject *newstr;
int changed; int changed;
WARN; WARN;
if (PyString_AsStringAndSize(args, &s, &n)) if (PyString_AsStringAndSize(args, &s, &n))
return NULL; return NULL;
new = PyString_FromStringAndSize(NULL, n); newstr = PyString_FromStringAndSize(NULL, n);
if (new == NULL) if (newstr == NULL)
return NULL; return NULL;
s_new = PyString_AsString(new); s_new = PyString_AsString(newstr);
changed = 0; changed = 0;
for (i = 0; i < n; i++) { for (i = 0; i < n; i++) {
int c = Py_CHARMASK(*s++); int c = Py_CHARMASK(*s++);
...@@ -467,11 +467,11 @@ strop_lower(PyObject *self, PyObject *args) ...@@ -467,11 +467,11 @@ strop_lower(PyObject *self, PyObject *args)
s_new++; s_new++;
} }
if (!changed) { if (!changed) {
Py_DECREF(new); Py_DECREF(newstr);
Py_INCREF(args); Py_INCREF(args);
return args; return args;
} }
return new; return newstr;
} }
...@@ -485,16 +485,16 @@ strop_upper(PyObject *self, PyObject *args) ...@@ -485,16 +485,16 @@ strop_upper(PyObject *self, PyObject *args)
{ {
char *s, *s_new; char *s, *s_new;
Py_ssize_t i, n; Py_ssize_t i, n;
PyObject *new; PyObject *newstr;
int changed; int changed;
WARN; WARN;
if (PyString_AsStringAndSize(args, &s, &n)) if (PyString_AsStringAndSize(args, &s, &n))
return NULL; return NULL;
new = PyString_FromStringAndSize(NULL, n); newstr = PyString_FromStringAndSize(NULL, n);
if (new == NULL) if (newstr == NULL)
return NULL; return NULL;
s_new = PyString_AsString(new); s_new = PyString_AsString(newstr);
changed = 0; changed = 0;
for (i = 0; i < n; i++) { for (i = 0; i < n; i++) {
int c = Py_CHARMASK(*s++); int c = Py_CHARMASK(*s++);
...@@ -506,11 +506,11 @@ strop_upper(PyObject *self, PyObject *args) ...@@ -506,11 +506,11 @@ strop_upper(PyObject *self, PyObject *args)
s_new++; s_new++;
} }
if (!changed) { if (!changed) {
Py_DECREF(new); Py_DECREF(newstr);
Py_INCREF(args); Py_INCREF(args);
return args; return args;
} }
return new; return newstr;
} }
...@@ -525,16 +525,16 @@ strop_capitalize(PyObject *self, PyObject *args) ...@@ -525,16 +525,16 @@ strop_capitalize(PyObject *self, PyObject *args)
{ {
char *s, *s_new; char *s, *s_new;
Py_ssize_t i, n; Py_ssize_t i, n;
PyObject *new; PyObject *newstr;
int changed; int changed;
WARN; WARN;
if (PyString_AsStringAndSize(args, &s, &n)) if (PyString_AsStringAndSize(args, &s, &n))
return NULL; return NULL;
new = PyString_FromStringAndSize(NULL, n); newstr = PyString_FromStringAndSize(NULL, n);
if (new == NULL) if (newstr == NULL)
return NULL; return NULL;
s_new = PyString_AsString(new); s_new = PyString_AsString(newstr);
changed = 0; changed = 0;
if (0 < n) { if (0 < n) {
int c = Py_CHARMASK(*s++); int c = Py_CHARMASK(*s++);
...@@ -555,11 +555,11 @@ strop_capitalize(PyObject *self, PyObject *args) ...@@ -555,11 +555,11 @@ strop_capitalize(PyObject *self, PyObject *args)
s_new++; s_new++;
} }
if (!changed) { if (!changed) {
Py_DECREF(new); Py_DECREF(newstr);
Py_INCREF(args); Py_INCREF(args);
return args; return args;
} }
return new; return newstr;
} }
...@@ -691,16 +691,16 @@ strop_swapcase(PyObject *self, PyObject *args) ...@@ -691,16 +691,16 @@ strop_swapcase(PyObject *self, PyObject *args)
{ {
char *s, *s_new; char *s, *s_new;
Py_ssize_t i, n; Py_ssize_t i, n;
PyObject *new; PyObject *newstr;
int changed; int changed;
WARN; WARN;
if (PyString_AsStringAndSize(args, &s, &n)) if (PyString_AsStringAndSize(args, &s, &n))
return NULL; return NULL;
new = PyString_FromStringAndSize(NULL, n); newstr = PyString_FromStringAndSize(NULL, n);
if (new == NULL) if (newstr == NULL)
return NULL; return NULL;
s_new = PyString_AsString(new); s_new = PyString_AsString(newstr);
changed = 0; changed = 0;
for (i = 0; i < n; i++) { for (i = 0; i < n; i++) {
int c = Py_CHARMASK(*s++); int c = Py_CHARMASK(*s++);
...@@ -717,11 +717,11 @@ strop_swapcase(PyObject *self, PyObject *args) ...@@ -717,11 +717,11 @@ strop_swapcase(PyObject *self, PyObject *args)
s_new++; s_new++;
} }
if (!changed) { if (!changed) {
Py_DECREF(new); Py_DECREF(newstr);
Py_INCREF(args); Py_INCREF(args);
return args; return args;
} }
return new; return newstr;
} }
...@@ -1141,7 +1141,7 @@ strop_replace(PyObject *self, PyObject *args) ...@@ -1141,7 +1141,7 @@ strop_replace(PyObject *self, PyObject *args)
char *str, *pat,*sub,*new_s; char *str, *pat,*sub,*new_s;
Py_ssize_t len,pat_len,sub_len,out_len; Py_ssize_t len,pat_len,sub_len,out_len;
Py_ssize_t count = -1; Py_ssize_t count = -1;
PyObject *new; PyObject *newstr;
WARN; WARN;
if (!PyArg_ParseTuple(args, "t#t#t#|n:replace", if (!PyArg_ParseTuple(args, "t#t#t#|n:replace",
...@@ -1165,14 +1165,14 @@ strop_replace(PyObject *self, PyObject *args) ...@@ -1165,14 +1165,14 @@ strop_replace(PyObject *self, PyObject *args)
} }
if (out_len == -1) { if (out_len == -1) {
/* we're returning another reference to the input string */ /* we're returning another reference to the input string */
new = PyTuple_GetItem(args, 0); newstr = PyTuple_GetItem(args, 0);
Py_XINCREF(new); Py_XINCREF(newstr);
} }
else { else {
new = PyString_FromStringAndSize(new_s, out_len); newstr = PyString_FromStringAndSize(new_s, out_len);
PyMem_FREE(new_s); PyMem_FREE(new_s);
} }
return new; return newstr;
} }
......
...@@ -443,7 +443,7 @@ time_strftime(PyObject *self, PyObject *args) ...@@ -443,7 +443,7 @@ time_strftime(PyObject *self, PyObject *args)
* will be ahead of time... * will be ahead of time...
*/ */
for (i = 1024; ; i += i) { for (i = 1024; ; i += i) {
outbuf = malloc(i); outbuf = (char *)malloc(i);
if (outbuf == NULL) { if (outbuf == NULL) {
return PyErr_NoMemory(); return PyErr_NoMemory();
} }
......
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