thread block/unblock in callback functions indentation update

Original commit message from CVS:
thread block/unblock in callback functions
indentation update
This commit is contained in:
David I. Lehn 2002-12-21 23:46:29 +00:00
parent b807857c46
commit 0b94dd64f0
3 changed files with 103 additions and 70 deletions

View file

@ -1,3 +1,10 @@
2002-12-21 David I. Lehn <dlehn@users.sourceforge.net>
* gstreamer/gstreamer.override (call_connect_function,
call_chain_function): block/unblock threads before entering Python
interpreter
* gstreamer/gstreamer.override: indent
2002-12-17 David I. Lehn <dlehn@users.sourceforge.net> 2002-12-17 David I. Lehn <dlehn@users.sourceforge.net>
* gstreamer/gstreamer.override: add wrapper for gst_element_set_state * gstreamer/gstreamer.override: add wrapper for gst_element_set_state

View file

@ -75,9 +75,12 @@ call_connect_function (GstPad *pad, GstCaps *caps)
{ {
PyObject *function; PyObject *function;
PyObject *retval; PyObject *retval;
GstPadConnectReturn ret;
function = pad_private(pad)->connect_function; function = pad_private(pad)->connect_function;
pyg_block_threads();
retval = (PyObject*)PyObject_CallFunction (function, retval = (PyObject*)PyObject_CallFunction (function,
"OO", "OO",
pad_private(pad)->pad, pad_private(pad)->pad,
@ -85,10 +88,15 @@ call_connect_function (GstPad *pad, GstCaps *caps)
if (PyErr_Occurred ()) { if (PyErr_Occurred ()) {
PyErr_Print (); PyErr_Print ();
pyg_unblock_threads();
return GST_PAD_CONNECT_REFUSED; return GST_PAD_CONNECT_REFUSED;
} }
return PyInt_AsLong(retval); ret = PyInt_AsLong(retval);
pyg_unblock_threads();
return ret;
} }
static PyObject* static PyObject*
@ -99,19 +107,19 @@ _wrap_gst_pad_set_connect_function (PyGObject *self,
static char *kwlist[] = { "connect_function", NULL }; static char *kwlist[] = { "connect_function", NULL };
PyObject *connect_function; PyObject *connect_function;
GstPad *pad; GstPad *pad;
if (!PyArg_ParseTupleAndKeywords(args, kwargs, if (!PyArg_ParseTupleAndKeywords(args, kwargs,
"O:GstPad.set_connect_funcion", "O:GstPad.set_connect_funcion",
kwlist, kwlist,
&connect_function)) { &connect_function)) {
return NULL; return NULL;
} }
if (!PyCallable_Check(connect_function)) { if (!PyCallable_Check(connect_function)) {
PyErr_SetString(PyExc_TypeError, "connect_function not callable"); PyErr_SetString(PyExc_TypeError, "connect_function not callable");
return NULL; return NULL;
} }
Py_INCREF(connect_function); Py_INCREF(connect_function);
py_pad_private(self)->connect_function = connect_function; py_pad_private(self)->connect_function = connect_function;
pad = (GstPad*)pygobject_get(self); pad = (GstPad*)pygobject_get(self);
@ -127,18 +135,23 @@ static void
call_chain_function(GstPad *pad, GstBuffer *buf) call_chain_function(GstPad *pad, GstBuffer *buf)
{ {
PyObject *function; PyObject *function;
function = pad_private(pad)->chain_function; function = pad_private(pad)->chain_function;
pyg_block_threads();
PyObject_CallFunction (function, PyObject_CallFunction (function,
"OO", "OO",
pad_private(pad)->pad, pad_private(pad)->pad,
pyg_boxed_new(GST_TYPE_BUFFER, buf, TRUE, TRUE)); pyg_boxed_new(GST_TYPE_BUFFER, buf, TRUE, TRUE));
if (PyErr_Occurred ()) { if (PyErr_Occurred ()) {
PyErr_Print (); PyErr_Print ();
pyg_unblock_threads();
return; return;
} }
pyg_unblock_threads();
} }
static PyObject* static PyObject*
@ -149,19 +162,19 @@ _wrap_gst_pad_set_chain_function(PyGObject *self,
static char *kwlist[] = { "chain_function", NULL }; static char *kwlist[] = { "chain_function", NULL };
PyObject *chain_function; PyObject *chain_function;
GstPad *pad; GstPad *pad;
if (!PyArg_ParseTupleAndKeywords(args, kwargs, if (!PyArg_ParseTupleAndKeywords(args, kwargs,
"O:GstPad.set_chain_funcion", "O:GstPad.set_chain_funcion",
kwlist, kwlist,
&chain_function)) { &chain_function)) {
return NULL; return NULL;
} }
if (!PyCallable_Check(chain_function)) { if (!PyCallable_Check(chain_function)) {
PyErr_SetString(PyExc_TypeError, "chain_function not callable"); PyErr_SetString(PyExc_TypeError, "chain_function not callable");
return NULL; return NULL;
} }
Py_INCREF(chain_function); Py_INCREF(chain_function);
py_pad_private(self)->chain_function = chain_function; py_pad_private(self)->chain_function = chain_function;
pad = (GstPad*)pygobject_get(self); pad = (GstPad*)pygobject_get(self);
@ -177,7 +190,7 @@ static PyObject*
_wrap_gst_buffer_get_data(PyObject *self) _wrap_gst_buffer_get_data(PyObject *self)
{ {
GstBuffer *buf; GstBuffer *buf;
buf = pyg_boxed_get(self, GstBuffer); buf = pyg_boxed_get(self, GstBuffer);
return PyString_FromStringAndSize( return PyString_FromStringAndSize(
@ -193,7 +206,7 @@ _wrap_gst_buffer_set_data(PyObject *self, PyObject *args, PyObject *kwargs)
static char *kwlist[] = {"data", NULL}; static char *kwlist[] = {"data", NULL};
PyObject *data; PyObject *data;
GstBuffer *buf; GstBuffer *buf;
if (!PyArg_ParseTupleAndKeywords(args, kwargs, "O:GstBuffer:set_data", kwlist, &data)) { if (!PyArg_ParseTupleAndKeywords(args, kwargs, "O:GstBuffer:set_data", kwlist, &data)) {
return NULL; return NULL;
} }
@ -222,12 +235,12 @@ override gst_bin_iterate
static PyObject * static PyObject *
_wrap_gst_bin_iterate(PyGObject *self) _wrap_gst_bin_iterate(PyGObject *self)
{ {
int ret; int ret;
pyg_unblock_threads(); pyg_unblock_threads();
ret = gst_bin_iterate(GST_BIN(self->obj)); ret = gst_bin_iterate(GST_BIN(self->obj));
pyg_block_threads(); pyg_block_threads();
return PyInt_FromLong(ret); return PyInt_FromLong(ret);
} }
%% %%
override gst_element_set_state kwargs override gst_element_set_state kwargs
@ -235,17 +248,17 @@ override gst_element_set_state kwargs
static PyObject * static PyObject *
_wrap_gst_element_set_state(PyGObject *self, PyObject *args, PyObject *kwargs) _wrap_gst_element_set_state(PyGObject *self, PyObject *args, PyObject *kwargs)
{ {
static char *kwlist[] = { "state", NULL }; static char *kwlist[] = { "state", NULL };
PyObject *py_state = NULL; PyObject *py_state = NULL;
GstElementState state; GstElementState state;
gint ret; gint ret;
if (!PyArg_ParseTupleAndKeywords(args, kwargs, "O:GstElement.set_state", kwlist, &py_state)) if (!PyArg_ParseTupleAndKeywords(args, kwargs, "O:GstElement.set_state", kwlist, &py_state))
return NULL; return NULL;
if (pyg_flags_get_value(GST_TYPE_ELEMENT_STATE, py_state, (gint *)&state)) if (pyg_flags_get_value(GST_TYPE_ELEMENT_STATE, py_state, (gint *)&state))
return NULL; return NULL;
pyg_unblock_threads(); pyg_unblock_threads();
ret = gst_element_set_state(GST_ELEMENT(self->obj), state); ret = gst_element_set_state(GST_ELEMENT(self->obj), state);
pyg_block_threads(); pyg_block_threads();
return PyInt_FromLong(ret); return PyInt_FromLong(ret);
} }

View file

@ -75,9 +75,12 @@ call_connect_function (GstPad *pad, GstCaps *caps)
{ {
PyObject *function; PyObject *function;
PyObject *retval; PyObject *retval;
GstPadConnectReturn ret;
function = pad_private(pad)->connect_function; function = pad_private(pad)->connect_function;
pyg_block_threads();
retval = (PyObject*)PyObject_CallFunction (function, retval = (PyObject*)PyObject_CallFunction (function,
"OO", "OO",
pad_private(pad)->pad, pad_private(pad)->pad,
@ -85,10 +88,15 @@ call_connect_function (GstPad *pad, GstCaps *caps)
if (PyErr_Occurred ()) { if (PyErr_Occurred ()) {
PyErr_Print (); PyErr_Print ();
pyg_unblock_threads();
return GST_PAD_CONNECT_REFUSED; return GST_PAD_CONNECT_REFUSED;
} }
return PyInt_AsLong(retval); ret = PyInt_AsLong(retval);
pyg_unblock_threads();
return ret;
} }
static PyObject* static PyObject*
@ -99,19 +107,19 @@ _wrap_gst_pad_set_connect_function (PyGObject *self,
static char *kwlist[] = { "connect_function", NULL }; static char *kwlist[] = { "connect_function", NULL };
PyObject *connect_function; PyObject *connect_function;
GstPad *pad; GstPad *pad;
if (!PyArg_ParseTupleAndKeywords(args, kwargs, if (!PyArg_ParseTupleAndKeywords(args, kwargs,
"O:GstPad.set_connect_funcion", "O:GstPad.set_connect_funcion",
kwlist, kwlist,
&connect_function)) { &connect_function)) {
return NULL; return NULL;
} }
if (!PyCallable_Check(connect_function)) { if (!PyCallable_Check(connect_function)) {
PyErr_SetString(PyExc_TypeError, "connect_function not callable"); PyErr_SetString(PyExc_TypeError, "connect_function not callable");
return NULL; return NULL;
} }
Py_INCREF(connect_function); Py_INCREF(connect_function);
py_pad_private(self)->connect_function = connect_function; py_pad_private(self)->connect_function = connect_function;
pad = (GstPad*)pygobject_get(self); pad = (GstPad*)pygobject_get(self);
@ -127,18 +135,23 @@ static void
call_chain_function(GstPad *pad, GstBuffer *buf) call_chain_function(GstPad *pad, GstBuffer *buf)
{ {
PyObject *function; PyObject *function;
function = pad_private(pad)->chain_function; function = pad_private(pad)->chain_function;
pyg_block_threads();
PyObject_CallFunction (function, PyObject_CallFunction (function,
"OO", "OO",
pad_private(pad)->pad, pad_private(pad)->pad,
pyg_boxed_new(GST_TYPE_BUFFER, buf, TRUE, TRUE)); pyg_boxed_new(GST_TYPE_BUFFER, buf, TRUE, TRUE));
if (PyErr_Occurred ()) { if (PyErr_Occurred ()) {
PyErr_Print (); PyErr_Print ();
pyg_unblock_threads();
return; return;
} }
pyg_unblock_threads();
} }
static PyObject* static PyObject*
@ -149,19 +162,19 @@ _wrap_gst_pad_set_chain_function(PyGObject *self,
static char *kwlist[] = { "chain_function", NULL }; static char *kwlist[] = { "chain_function", NULL };
PyObject *chain_function; PyObject *chain_function;
GstPad *pad; GstPad *pad;
if (!PyArg_ParseTupleAndKeywords(args, kwargs, if (!PyArg_ParseTupleAndKeywords(args, kwargs,
"O:GstPad.set_chain_funcion", "O:GstPad.set_chain_funcion",
kwlist, kwlist,
&chain_function)) { &chain_function)) {
return NULL; return NULL;
} }
if (!PyCallable_Check(chain_function)) { if (!PyCallable_Check(chain_function)) {
PyErr_SetString(PyExc_TypeError, "chain_function not callable"); PyErr_SetString(PyExc_TypeError, "chain_function not callable");
return NULL; return NULL;
} }
Py_INCREF(chain_function); Py_INCREF(chain_function);
py_pad_private(self)->chain_function = chain_function; py_pad_private(self)->chain_function = chain_function;
pad = (GstPad*)pygobject_get(self); pad = (GstPad*)pygobject_get(self);
@ -177,7 +190,7 @@ static PyObject*
_wrap_gst_buffer_get_data(PyObject *self) _wrap_gst_buffer_get_data(PyObject *self)
{ {
GstBuffer *buf; GstBuffer *buf;
buf = pyg_boxed_get(self, GstBuffer); buf = pyg_boxed_get(self, GstBuffer);
return PyString_FromStringAndSize( return PyString_FromStringAndSize(
@ -193,7 +206,7 @@ _wrap_gst_buffer_set_data(PyObject *self, PyObject *args, PyObject *kwargs)
static char *kwlist[] = {"data", NULL}; static char *kwlist[] = {"data", NULL};
PyObject *data; PyObject *data;
GstBuffer *buf; GstBuffer *buf;
if (!PyArg_ParseTupleAndKeywords(args, kwargs, "O:GstBuffer:set_data", kwlist, &data)) { if (!PyArg_ParseTupleAndKeywords(args, kwargs, "O:GstBuffer:set_data", kwlist, &data)) {
return NULL; return NULL;
} }
@ -222,12 +235,12 @@ override gst_bin_iterate
static PyObject * static PyObject *
_wrap_gst_bin_iterate(PyGObject *self) _wrap_gst_bin_iterate(PyGObject *self)
{ {
int ret; int ret;
pyg_unblock_threads(); pyg_unblock_threads();
ret = gst_bin_iterate(GST_BIN(self->obj)); ret = gst_bin_iterate(GST_BIN(self->obj));
pyg_block_threads(); pyg_block_threads();
return PyInt_FromLong(ret); return PyInt_FromLong(ret);
} }
%% %%
override gst_element_set_state kwargs override gst_element_set_state kwargs
@ -235,17 +248,17 @@ override gst_element_set_state kwargs
static PyObject * static PyObject *
_wrap_gst_element_set_state(PyGObject *self, PyObject *args, PyObject *kwargs) _wrap_gst_element_set_state(PyGObject *self, PyObject *args, PyObject *kwargs)
{ {
static char *kwlist[] = { "state", NULL }; static char *kwlist[] = { "state", NULL };
PyObject *py_state = NULL; PyObject *py_state = NULL;
GstElementState state; GstElementState state;
gint ret; gint ret;
if (!PyArg_ParseTupleAndKeywords(args, kwargs, "O:GstElement.set_state", kwlist, &py_state)) if (!PyArg_ParseTupleAndKeywords(args, kwargs, "O:GstElement.set_state", kwlist, &py_state))
return NULL; return NULL;
if (pyg_flags_get_value(GST_TYPE_ELEMENT_STATE, py_state, (gint *)&state)) if (pyg_flags_get_value(GST_TYPE_ELEMENT_STATE, py_state, (gint *)&state))
return NULL; return NULL;
pyg_unblock_threads(); pyg_unblock_threads();
ret = gst_element_set_state(GST_ELEMENT(self->obj), state); ret = gst_element_set_state(GST_ELEMENT(self->obj), state);
pyg_block_threads(); pyg_block_threads();
return PyInt_FromLong(ret); return PyInt_FromLong(ret);
} }