Mercurial > cpython-lru_cache-weakref
diff Modules/threadmodule.c @ 22709:3aa4bf7ac31e legacy-trunk
Remove METH_OLDARGS:
Convert METH_OLDARGS -> METH_VARARGS: also PyArg_Parse -> PyArg_ParseTuple
Convert METH_OLDARGS -> METH_NOARGS: remove args parameter
Please review. All tests pass, but some modules don't have tests.
I spot checked various functions to try to make sure nothing broke.
| author | Neal Norwitz <nnorwitz@gmail.com> |
|---|---|
| date | Sun, 31 Mar 2002 15:27:00 +0000 |
| parents | bcd1fdf13acf |
| children | 7014921820a5 |
line wrap: on
line diff
--- a/Modules/threadmodule.c Sun Mar 31 14:57:24 2002 +0000 +++ b/Modules/threadmodule.c Sun Mar 31 15:27:00 2002 +0000 @@ -54,14 +54,10 @@ static PyObject * lock_PyThread_acquire_lock(lockobject *self, PyObject *args) { - int i; + int i = 1; - if (args != NULL) { - if (!PyArg_Parse(args, "i", &i)) - return NULL; - } - else - i = 1; + if (!PyArg_ParseTuple(args, "|i:acquire", &i)) + return NULL; Py_BEGIN_ALLOW_THREADS i = PyThread_acquire_lock(self->lock_lock, i); @@ -127,9 +123,9 @@ static PyMethodDef lock_methods[] = { {"acquire_lock", (PyCFunction)lock_PyThread_acquire_lock, - METH_OLDARGS, acquire_doc}, + METH_VARARGS, acquire_doc}, {"acquire", (PyCFunction)lock_PyThread_acquire_lock, - METH_OLDARGS, acquire_doc}, + METH_VARARGS, acquire_doc}, {"release_lock", (PyCFunction)lock_PyThread_release_lock, METH_NOARGS, release_doc}, {"release", (PyCFunction)lock_PyThread_release_lock, @@ -279,7 +275,7 @@ thread_PyThread_exit_prog(PyObject *self, PyObject *args) { int sts; - if (!PyArg_Parse(args, "i", &sts)) + if (!PyArg_ParseTuple(args, "i:exit_prog", &sts)) return NULL; Py_Exit(sts); /* Calls PyThread_exit_prog(sts) or _PyThread_exit_prog(sts) */ for (;;) { } /* Should not be reached */ @@ -339,7 +335,8 @@ {"get_ident", (PyCFunction)thread_get_ident, METH_NOARGS, get_ident_doc}, #ifndef NO_EXIT_PROG - {"exit_prog", (PyCFunction)thread_PyThread_exit_prog}, + {"exit_prog", (PyCFunction)thread_PyThread_exit_prog, + METH_VARARGS}, #endif {NULL, NULL} /* sentinel */ };