Skip to content

Commit

Permalink
Simplify getbuffer(): convertbuffer() fails anyway if bf_getbuffer is…
Browse files Browse the repository at this point in the history
… NULL
  • Loading branch information
Victor Stinner committed Jun 6, 2010
1 parent 7047eb7 commit 5cb6239
Showing 1 changed file with 11 additions and 19 deletions.
30 changes: 11 additions & 19 deletions Python/getargs.c
Original file line number Diff line number Diff line change
Expand Up @@ -1410,7 +1410,7 @@ convertsimple(PyObject *arg, const char **p_format, va_list *p_va, int flags,
static Py_ssize_t
convertbuffer(PyObject *arg, void **p, char **errmsg)
{
PyBufferProcs *pb = arg->ob_type->tp_as_buffer;
PyBufferProcs *pb = Py_TYPE(arg)->tp_as_buffer;
Py_ssize_t count;
Py_buffer view;

Expand Down Expand Up @@ -1438,31 +1438,23 @@ convertbuffer(PyObject *arg, void **p, char **errmsg)
static int
getbuffer(PyObject *arg, Py_buffer *view, char **errmsg)
{
void *buf;
Py_ssize_t count;
PyBufferProcs *pb = arg->ob_type->tp_as_buffer;
PyBufferProcs *pb = Py_TYPE(arg)->tp_as_buffer;
if (pb == NULL) {
*errmsg = "bytes or buffer";
return -1;
}
if (pb->bf_getbuffer) {
if (PyObject_GetBuffer(arg, view, 0) < 0) {
*errmsg = "convertible to a buffer";
return -1;
}
if (!PyBuffer_IsContiguous(view, 'C')) {
*errmsg = "contiguous buffer";
return -1;
}
return 0;
if (pb->bf_getbuffer == NULL) {
*errmsg = "convertible to a buffer";
return -1;
}

count = convertbuffer(arg, &buf, errmsg);
if (count < 0) {
if (PyObject_GetBuffer(arg, view, 0) < 0) {
*errmsg = "convertible to a buffer";
return count;
return -1;
}
if (!PyBuffer_IsContiguous(view, 'C')) {
*errmsg = "contiguous buffer";
return -1;
}
PyBuffer_FillInfo(view, NULL, buf, count, 1, 0);
return 0;
}

Expand Down

0 comments on commit 5cb6239

Please sign in to comment.