X-Git-Url: https://sigrok.org/gitweb/?p=libsigrokdecode.git;a=blobdiff_plain;f=type_decoder.c;h=8395edd4d21671be3127ad2c5232c4a440b543f6;hp=86ccf02ea7fcd46a3d7626b7f8992e2695962adf;hb=fa12a21e3c779df4e33dcfe394e2ded3f96028ce;hpb=2086c68463fef416fd9a48dac5e1aa7fa593308d

diff --git a/type_decoder.c b/type_decoder.c
index 86ccf02..8395edd 100644
--- a/type_decoder.c
+++ b/type_decoder.c
@@ -20,6 +20,15 @@
 #include "sigrokdecode.h" /* First, so we avoid a _POSIX_C_SOURCE warning. */
 #include "sigrokdecode-internal.h"
 #include "config.h"
+#include <inttypes.h>
+
+
+/* This is only used for nicer srd_dbg() output. */
+char *OUTPUT_TYPES[] = {
+	"OUTPUT_ANN",
+	"OUTPUT_PROTO",
+	"OUTPUT_BINARY",
+};
 
 
 static int convert_pyobj(struct srd_decoder_instance *di, PyObject *obj,
@@ -87,10 +96,16 @@ static PyObject *Decoder_put(PyObject *self, PyObject *args)
 	int output_id;
 	void (*cb)();
 
-	if (!(di = get_di_by_decobject(self)))
+	if (!(di = srd_instance_find_by_obj(NULL, self))) {
+		/* Shouldn't happen. */
+		srd_dbg("put(): self instance not found.");
 		return NULL;
+	}
 
 	if (!PyArg_ParseTuple(args, "KKiO", &start_sample, &end_sample, &output_id, &data))
+		/* This throws an exception, but by returning NULL here we let python
+		 * raise it. This results in a much better trace in controller.c
+		 * on the decode() method call. */
 		return NULL;
 
 	if (!(l = g_slist_nth(di->pd_output, output_id))) {
@@ -100,6 +115,10 @@ static PyObject *Decoder_put(PyObject *self, PyObject *args)
 	}
 	pdo = l->data;
 
+	srd_spew("Instance %s put %" PRIu64 "-%" PRIu64 " %s on oid %d.",
+		 di->instance_id, start_sample, end_sample,
+		 OUTPUT_TYPES[pdo->output_type], output_id);
+
 	if (!(pdata = g_try_malloc0(sizeof(struct srd_proto_data))))
 		return NULL;
 	pdata->start_sample = start_sample;
@@ -124,6 +143,8 @@ static PyObject *Decoder_put(PyObject *self, PyObject *args)
 			next_di = l->data;
 			/* TODO: is this needed? */
 			Py_XINCREF(next_di->py_instance);
+			srd_spew("Sending %d-%d to instance %s",
+				 start_sample, end_sample, next_di->instance_id);
 			if (!(py_res = PyObject_CallMethod(next_di->py_instance, "decode",
 					"KKO", start_sample, end_sample, data))) {
 				catch_exception("calling %s decode(): ", next_di->instance_id);
@@ -153,14 +174,13 @@ static PyObject *Decoder_add(PyObject *self, PyObject *args)
 	char *proto_id;
 	int output_type, pdo_id;
 
-	if (!(di = get_di_by_decobject(self))) {
-		srd_dbg("srd: decoder instance not found");
+	if (!(di = srd_instance_find_by_obj(NULL, self))) {
 		PyErr_SetString(PyExc_Exception, "decoder instance not found");
 		return NULL;
 	}
 
 	if (!PyArg_ParseTuple(args, "is", &output_type, &proto_id)) {
-		catch_exception("");
+		/* Let python raise this exception. */
 		return NULL;
 	}