Add an optional 'offset' parameter to byref, defaultingto zero.

diff --git a/Modules/_ctypes/callproc.c b/Modules/_ctypes/callproc.c
index 825a2c742344d70eb9429db7e67d423330a25a5d..d2727e49ea11ff001f58f0efc71854c316e91c84 100644 (file)
--- a/Modules/_ctypes/callproc.c
+++ b/Modules/_ctypes/callproc.c
@@ -1561,7 +1561,7 @@ align_func(PyObject *self, PyObject *obj)
 }
 
 static char byref_doc[] =
-"byref(C instance) -> byref-object\n"
+"byref(C instance[, offset=0]) -> byref-object\n"
 "Return a pointer lookalike to a C instance, only usable\n"
 "as function argument";
 
@@ -1570,9 +1570,21 @@ static char byref_doc[] =
  * but still has a reference to self.
  */
 static PyObject *
-byref(PyObject *self, PyObject *obj)
+byref(PyObject *self, PyObject *args)
 {
        PyCArgObject *parg;
+       PyObject *obj;
+       PyObject *pyoffset = NULL;
+       Py_ssize_t offset = 0;
+
+       if (!PyArg_UnpackTuple(args, "byref", 1, 2,
+                              &obj, &pyoffset))
+               return NULL;
+       if (pyoffset) {
+               offset = PyNumber_AsSsize_t(pyoffset, NULL);
+               if (offset == -1 && PyErr_Occurred())
+                       return NULL;
+       }
        if (!CDataObject_Check(obj)) {
                PyErr_Format(PyExc_TypeError,
                             "byref() argument must be a ctypes instance, not '%s'",
@@ -1588,7 +1600,7 @@ byref(PyObject *self, PyObject *obj)
        parg->pffi_type = &ffi_type_pointer;
        Py_INCREF(obj);
        parg->obj = obj;
-       parg->value.p = ((CDataObject *)obj)->b_ptr;
+       parg->value.p = (char *)((CDataObject *)obj)->b_ptr + offset;
        return (PyObject *)parg;
 }
 
@@ -1876,7 +1888,7 @@ PyMethodDef module_methods[] = {
 #endif
        {"alignment", align_func, METH_O, alignment_doc},
        {"sizeof", sizeof_func, METH_O, sizeof_doc},
-       {"byref", byref, METH_O, byref_doc},
+       {"byref", byref, METH_VARARGS, byref_doc},
        {"addressof", addressof, METH_O, addressof_doc},
        {"call_function", call_function, METH_VARARGS },
        {"call_cdeclfunction", call_cdeclfunction, METH_VARARGS },