Simplify code.

This commit is contained in:
Anthony Tuininga 2018-02-16 16:20:29 -07:00
parent ce560d7330
commit 0d3aa46d02
2 changed files with 6 additions and 21 deletions

View File

@ -89,7 +89,7 @@ static PyMethodDef cxoConnectionMethods[] = {
METH_VARARGS | METH_KEYWORDS},
{ "changepassword", (PyCFunction) cxoConnection_changePassword,
METH_VARARGS },
{ "gettype", (PyCFunction) cxoConnection_getType, METH_VARARGS },
{ "gettype", (PyCFunction) cxoConnection_getType, METH_O },
{ "deqoptions", (PyCFunction) cxoConnection_newDequeueOptions,
METH_NOARGS },
{ "enqoptions", (PyCFunction) cxoConnection_newEnqueueOptions,
@ -895,14 +895,8 @@ static int cxoConnection_setStmtCacheSize(cxoConnection* conn, PyObject *value,
// cxoConnection_getType()
// Return a type object given its name.
//-----------------------------------------------------------------------------
static PyObject *cxoConnection_getType(cxoConnection *conn, PyObject *args)
static PyObject *cxoConnection_getType(cxoConnection *conn, PyObject *nameObj)
{
PyObject *nameObj = NULL;
// parse the arguments
if (!PyArg_ParseTuple(args, "O", &nameObj))
return NULL;
return (PyObject*) cxoObjectType_newByName(conn, nameObj);
}

View File

@ -41,12 +41,12 @@ static PyObject *cxoObject_trim(cxoObject*, PyObject*);
// declaration of methods for Python type "Object"
//-----------------------------------------------------------------------------
static PyMethodDef cxoObjectMethods[] = {
{ "append", (PyCFunction) cxoObject_append, METH_VARARGS },
{ "append", (PyCFunction) cxoObject_append, METH_O },
{ "aslist", (PyCFunction) cxoObject_asList, METH_NOARGS },
{ "copy", (PyCFunction) cxoObject_copy, METH_NOARGS },
{ "delete", (PyCFunction) cxoObject_delete, METH_VARARGS },
{ "exists", (PyCFunction) cxoObject_exists, METH_VARARGS },
{ "extend", (PyCFunction) cxoObject_extend, METH_VARARGS },
{ "extend", (PyCFunction) cxoObject_extend, METH_O },
{ "first", (PyCFunction) cxoObject_getFirstIndex, METH_NOARGS },
{ "getelement", (PyCFunction) cxoObject_getElement, METH_VARARGS },
{ "last", (PyCFunction) cxoObject_getLastIndex, METH_NOARGS },
@ -336,15 +336,10 @@ int cxoObject_internalExtend(cxoObject *obj, PyObject *sequence)
// cxoObject_append()
// Append an item to the collection.
//-----------------------------------------------------------------------------
static PyObject *cxoObject_append(cxoObject *obj, PyObject *args)
static PyObject *cxoObject_append(cxoObject *obj, PyObject *value)
{
PyObject *value;
if (!PyArg_ParseTuple(args, "O", &value))
return NULL;
if (cxoObject_internalAppend(obj, value) < 0)
return NULL;
Py_RETURN_NONE;
}
@ -480,12 +475,8 @@ static PyObject *cxoObject_exists(cxoObject *obj, PyObject *args)
// cxoObject_extend()
// Extend the collection by appending each of the items in the sequence.
//-----------------------------------------------------------------------------
static PyObject *cxoObject_extend(cxoObject *obj, PyObject *args)
static PyObject *cxoObject_extend(cxoObject *obj, PyObject *sequence)
{
PyObject *sequence;
if (!PyArg_ParseTuple(args, "O", &sequence))
return NULL;
if (cxoObject_internalExtend(obj, sequence) < 0)
return NULL;
Py_RETURN_NONE;