rfm12: Avoid using the Python 'range' keyword as variable.

[libsigrokdecode.git] / type_decoder.c

diff --git a/type_decoder.c b/type_decoder.c
index 6810d919043bc61337aa0f6b936fa4eefdc34d90..a7b23de996a19eb0fbe8bd92caa489a4c958e608 100644 (file)
--- a/type_decoder.c
+++ b/type_decoder.c
@@ -17,11 +17,15 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include "libsigrokdecode.h" /* First, so we avoid a _POSIX_C_SOURCE warning. */
-#include "libsigrokdecode-internal.h"
+#include "libsigrokdecode-internal.h" /* First, so we avoid a _POSIX_C_SOURCE warning. */
+#include "libsigrokdecode.h"
 #include "config.h"
 #include <inttypes.h>
 
+typedef struct {
+        PyObject_HEAD
+} srd_Decoder;
+
 /* This is only used for nicer srd_dbg() output. */
 static const char *OUTPUT_TYPES[] = {
        "OUTPUT_ANN",
@@ -209,7 +213,7 @@ static PyObject *Decoder_put(PyObject *self, PyObject *args)
        }
 
        if (!PyArg_ParseTuple(args, "KKiO", &start_sample, &end_sample,
-           &output_id, &py_data)) {
+               &output_id, &py_data)) {
                /*
                 * This throws an exception, but by returning NULL here we let
                 * Python raise it. This results in a much better trace in
@@ -253,16 +257,21 @@ static PyObject *Decoder_put(PyObject *self, PyObject *args)
                for (l = di->next_di; l; l = l->next) {
                        next_di = l->data;
                        srd_spew("Sending %d-%d to instance %s",
-                                start_sample, end_sample,
-                                next_di->inst_id);
+                                start_sample, end_sample, next_di->inst_id);
                        if (!(py_res = PyObject_CallMethod(
-                           next_di->py_inst, "decode", "KKO", start_sample,
-                           end_sample, py_data))) {
+                               next_di->py_inst, "decode", "KKO", start_sample,
+                               end_sample, py_data))) {
                                srd_exception_catch("Calling %s decode(): ",
-                                                   next_di->inst_id);
+                                                       next_di->inst_id);
                        }
                        Py_XDECREF(py_res);
                }
+               if ((cb = srd_pd_output_callback_find(di->sess, pdo->output_type))) {
+                       /* Frontends aren't really supposed to get Python
+                        * callbacks, but it's useful for testing. */
+                       pdata->data = py_data;
+                       cb->cb(pdata, cb->cb_data);
+               }
                break;
        case SRD_OUTPUT_BINARY:
                if ((cb = srd_pd_output_callback_find(di->sess, pdo->output_type))) {
@@ -364,23 +373,9 @@ static PyObject *Decoder_register(PyObject *self, PyObject *args,
        return py_new_output_id;
 }
 
-/* TODO: this is just a stub that calls _register() until all PDs
- * are changed to use the new register API. */
-static PyObject *Decoder_add(PyObject *self, PyObject *args)
-{
-       PyObject *py_keywords, *py_new_output_id;
-
-       py_keywords = PyDict_New();
-       py_new_output_id = Decoder_register(self, args, py_keywords);
-       Py_DecRef(py_keywords);
-
-       return py_new_output_id;
-}
-
 static PyMethodDef Decoder_methods[] = {
        {"put", Decoder_put, METH_VARARGS,
         "Accepts a dictionary with the following keys: startsample, endsample, data"},
-       {"add", Decoder_add, METH_VARARGS, "Create a new output stream"},
        {"register", (PyCFunction)Decoder_register, METH_VARARGS|METH_KEYWORDS,
                        "Register a new output stream"},
        {NULL, NULL, 0, NULL}