[ctypes-commit] ctypes/source callproc.c,1.144,1.145

[Thomas H. <th...@us...>]

Update of /cvsroot/ctypes/ctypes/source
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv10425

Modified Files:
	callproc.c 
Log Message:
Simpler and better code for PyObject_asparam.


Index: callproc.c
===================================================================
RCS file: /cvsroot/ctypes/ctypes/source/callproc.c,v
retrieving revision 1.144
retrieving revision 1.145
diff -C2 -d -r1.144 -r1.145
*** callproc.c	7 Apr 2005 18:43:44 -0000	1.144
--- callproc.c	7 Apr 2005 20:31:26 -0000	1.145
***************
*** 395,427 ****
  	if (stgdict)
  		return stgdict->asparam((CDataObject *)obj, pa);
! 
! 	/* probably a byref(obj) parameter */
! 	if (PyCArg_CheckExact(obj)) {
! 		PyCArgObject *carg = (PyCArgObject *)obj;
! 		pa->ffi_type = carg->pffi_type;
! 		Py_INCREF(obj);
! 		pa->keep = obj;
! 		memcpy(&pa->value, &carg->value, sizeof(pa->value));
  		return 0;
  	}
! 
! 	/* Pass as integer by calling i_set() */
! 	if (PyInt_Check(obj) || PyLong_Check(obj)) {
  		pa->ffi_type = &ffi_type_sint;
! 		pa->keep = getentry("i")->setfunc(&pa->value, obj, 0, NULL); /* CTYPE_c_int? */
! 		if (pa->keep == NULL)
! 			return -1;
  		return 0;
  	}
! 
! 	/* Pass as pointer by calling z_set() */
! 	if (obj == Py_None || PyString_Check(obj)) {
  		pa->ffi_type = &ffi_type_pointer;
! 		pa->keep = getentry("z")->setfunc(&pa->value, obj, 0, NULL); /* CTYPE_c_char_p? */
! 		if (pa->keep == NULL)
! 			return -1;
  		return 0;
  	}
  #ifdef CTYPES_UNICODE
  	/* Pass as pointer by calling Z_set() */
  	if (PyUnicode_Check(obj)) {
--- 395,423 ----
  	if (stgdict)
  		return stgdict->asparam((CDataObject *)obj, pa);
! 	if (obj == Py_None) {
! 		pa->ffi_type = &ffi_type_pointer;
! 		pa->value.p = NULL;
  		return 0;
  	}
! 	if (PyInt_Check(obj)) {
  		pa->ffi_type = &ffi_type_sint;
! 		pa->value.i = PyInt_AS_LONG(obj);
  		return 0;
  	}
! 	if (PyLong_Check(obj)) {
! 		pa->ffi_type = &ffi_type_sint;
! 		pa->value.i = (int)PyLong_AsUnsignedLongMask(obj);
! 		return 0;
! 	}
! 	if (PyString_Check(obj)) {
  		pa->ffi_type = &ffi_type_pointer;
! 		pa->value.p = PyString_AS_STRING(obj);
! 		Py_INCREF(obj);
! 		pa->keep = obj;
  		return 0;
  	}
  #ifdef CTYPES_UNICODE
+ 	/* XXX See Z_set. */
+ 	/* XXX PyUnicode_AsWideChar(), */
  	/* Pass as pointer by calling Z_set() */
  	if (PyUnicode_Check(obj)) {
***************
*** 433,436 ****
--- 429,440 ----
  	}
  #endif
+ 	if (PyCArg_CheckExact(obj)) {
+ 		PyCArgObject *carg = (PyCArgObject *)obj;
+ 		pa->ffi_type = carg->pffi_type;
+ 		Py_INCREF(obj);
+ 		pa->keep = obj;
+ 		memcpy(&pa->value, &carg->value, sizeof(pa->value));
+ 		return 0;
+ 	}
  
  	arg = PyObject_GetAttrString(obj, "_as_parameter_");