Make new gcc -Wall happy

This commit is contained in:
Guido van Rossum 1998-04-10 22:16:39 +00:00
parent bb3649e2ba
commit 1109fbca76
4 changed files with 13 additions and 7 deletions

View File

@ -190,10 +190,12 @@ file_dealloc(f)
(*f->f_close)(f->f_fp);
Py_END_ALLOW_THREADS
}
if (f->f_name != NULL)
if (f->f_name != NULL) {
Py_DECREF(f->f_name);
if (f->f_mode != NULL)
}
if (f->f_mode != NULL) {
Py_DECREF(f->f_mode);
}
free((char *)f);
}
@ -771,8 +773,9 @@ file_readlines(f, args)
goto error;
}
cleanup:
if (big_buffer)
if (big_buffer) {
Py_DECREF(big_buffer);
}
return list;
}

View File

@ -110,8 +110,9 @@ PyFunction_SetDefaults(op, defaults)
}
if (defaults == Py_None)
defaults = NULL;
else if (PyTuple_Check(defaults))
else if (PyTuple_Check(defaults)) {
Py_XINCREF(defaults);
}
else {
PyErr_SetString(PyExc_SystemError, "non-tuple default args");
return -1;

View File

@ -497,7 +497,7 @@ PyLong_FromString(str, pend, base)
static PyLongObject *x_divrem
Py_PROTO((PyLongObject *, PyLongObject *, PyLongObject **));
static PyObject *long_pos Py_PROTO((PyLongObject *));
static long_divrem Py_PROTO((PyLongObject *, PyLongObject *,
static int long_divrem Py_PROTO((PyLongObject *, PyLongObject *,
PyLongObject **, PyLongObject **));
/* Long division with remainder, top-level routine */

View File

@ -1016,14 +1016,16 @@ PyString_Format(format, args)
"not all arguments converted");
goto error;
}
if (args_owned)
if (args_owned) {
Py_DECREF(args);
}
_PyString_Resize(&result, reslen - rescnt);
return result;
error:
Py_DECREF(result);
if (args_owned)
if (args_owned) {
Py_DECREF(args);
}
return NULL;
}