static int convert_pyobj(struct srd_decoder_instance *di, PyObject *obj,
- int *annotation_format, char ***annotation)
+ int *ann_format, char ***ann)
{
PyObject *py_tmp;
struct srd_pd_output *pdo;
}
ann_id = PyLong_AsLong(py_tmp);
- if (!(pdo = g_slist_nth_data(di->decoder->annotation, ann_id))) {
+ if (!(pdo = g_slist_nth_data(di->decoder->annotations, ann_id))) {
srd_err("Protocol decoder %s submitted data to non-existent annotation format %d",
di->decoder->name, ann_id);
return SRD_ERR_PYTHON;
}
- *annotation_format = ann_id;
+ *ann_format = ann_id;
/* Second element must be a list */
py_tmp = PyList_GetItem(obj, 1);
di->decoder->name);
return SRD_ERR_PYTHON;
}
- if (py_strlist_to_char(py_tmp, annotation) != SRD_OK) {
+ if (py_strlist_to_char(py_tmp, ann) != SRD_OK) {
srd_err("Protocol decoder %s submitted annotation list, but second element was malformed",
di->decoder->name);
return SRD_ERR_PYTHON;
return SRD_OK;
}
-/* TODO: not used, doesn't work actually */
-static PyObject *Decoder_init(PyObject *self, PyObject *args)
-{
- (void)self;
- (void)args;
- printf("init Decoder object %p\n", self);
-
- Py_RETURN_NONE;
-}
-
static PyObject *Decoder_put(PyObject *self, PyObject *args)
{
GSList *l;
- PyObject *data;
- struct srd_decoder_instance *di;
+ PyObject *data, *py_res;
+ struct srd_decoder_instance *di, *next_di;
struct srd_pd_output *pdo;
- struct srd_protocol_data *pdata;
+ struct srd_proto_data *pdata;
uint64_t start_sample, end_sample;
int output_id;
void (*cb)();
}
pdo = l->data;
+ if (!(pdata = g_try_malloc0(sizeof(struct srd_proto_data))))
+ return NULL;
+ pdata->start_sample = start_sample;
+ pdata->end_sample = end_sample;
+ pdata->pdo = pdo;
+
switch (pdo->output_type) {
- case SRD_OUTPUT_ANNOTATION:
- case SRD_OUTPUT_PROTOCOL:
+ case SRD_OUTPUT_ANN:
+ /* Annotations are only fed to callbacks. */
+ if ((cb = srd_find_callback(pdo->output_type))) {
+ /* Annotations need converting from PyObject. */
+ if (convert_pyobj(di, data, &pdata->ann_format,
+ (char ***)&pdata->data) != SRD_OK) {
+ /* An error was already logged. */
+ break;
+ }
+ cb(pdata);
+ }
+ break;
+ case SRD_OUTPUT_PROTO:
+ for (l = di->next_di; l; l = l->next) {
+ next_di = l->data;
+ /* TODO: is this needed? */
+ Py_XINCREF(next_di->py_instance);
+ if (!(py_res = PyObject_CallMethod(next_di->py_instance, "decode",
+ "KKO", start_sample, end_sample, data))) {
+ if (PyErr_Occurred())
+ PyErr_Print();
+ }
+ Py_XDECREF(py_res);
+ }
+ break;
case SRD_OUTPUT_BINARY:
+ srd_err("SRD_OUTPUT_BINARY not yet supported");
break;
default:
srd_err("Protocol decoder %s submitted invalid output type %d",
di->decoder->name, pdo->output_type);
- return NULL;
break;
}
- if ((cb = srd_find_callback(pdo->output_type))) {
- /* Something registered an interest in this output type. */
- if (!(pdata = g_try_malloc0(sizeof(struct srd_protocol_data))))
- return NULL;
- pdata->start_sample = start_sample;
- pdata->end_sample = end_sample;
- pdata->pdo = pdo;
- if (pdo->output_type == SRD_OUTPUT_ANNOTATION) {
- /* annotations need converting from PyObject */
- if (convert_pyobj(di, data, &pdata->annotation_format,
- (char ***)&pdata->data) != SRD_OK)
- return NULL;
- } else {
- /* annotation_format is unused, data is an opaque blob. */
- pdata->data = data;
- }
- cb(pdata);
- }
+ g_free(pdata);
Py_RETURN_NONE;
}
{
PyObject *ret;
struct srd_decoder_instance *di;
- char *protocol_id;
+ char *proto_id;
int output_type, pdo_id;
- if (!(di = get_di_by_decobject(self)))
+ if (!(di = get_di_by_decobject(self))) {
+ srd_err("%s():%d decoder instance not found", __func__, __LINE__);
+ PyErr_SetString(PyExc_Exception, "decoder instance not found");
return NULL;
+ }
- printf("output_new di %s\n", di->decoder->name);
-
- if (!PyArg_ParseTuple(args, "is", &output_type, &protocol_id))
+ if (!PyArg_ParseTuple(args, "is", &output_type, &proto_id)) {
+ if (PyErr_Occurred())
+ PyErr_Print();
return NULL;
+ }
- pdo_id = pd_add(di, output_type, protocol_id);
+ pdo_id = pd_add(di, output_type, proto_id);
if (pdo_id < 0)
Py_RETURN_NONE;
else
.tp_flags = Py_TPFLAGS_DEFAULT | Py_TPFLAGS_BASETYPE,
.tp_doc = "Sigrok Decoder object",
.tp_methods = Decoder_methods,
- .tp_init = (initproc) Decoder_init,
};
static struct PyModuleDef sigrokdecode_module = {
return NULL;
/* expose output types as symbols in the sigrokdecode module */
- if(PyModule_AddObject(mod, "SRD_OUTPUT_ANNOTATION",
- PyLong_FromLong(SRD_OUTPUT_ANNOTATION)) == -1)
+ if(PyModule_AddObject(mod, "OUTPUT_ANN",
+ PyLong_FromLong(SRD_OUTPUT_ANN)) == -1)
return NULL;
- if(PyModule_AddObject(mod, "SRD_OUTPUT_PROTOCOL",
- PyLong_FromLong(SRD_OUTPUT_PROTOCOL)) == -1)
+ if(PyModule_AddObject(mod, "OUTPUT_PROTO",
+ PyLong_FromLong(SRD_OUTPUT_PROTO)) == -1)
return NULL;
- if(PyModule_AddObject(mod, "SRD_OUTPUT_BINARY",
+ if(PyModule_AddObject(mod, "OUTPUT_BINARY",
PyLong_FromLong(SRD_OUTPUT_BINARY)) == -1)
return NULL;