Python: Restrict code to stable ABI subset

[libsigrokdecode.git] / type_decoder.c

diff --git a/type_decoder.c b/type_decoder.c
index d4c592f224e8c3f1b933cf0fb1ac5d6f981e47a8..89b5ca1119c439d5caecd1279c2272dae907f413 100644 (file)
--- a/type_decoder.c
+++ b/type_decoder.c
@@ -26,13 +26,19 @@ typedef struct {
         PyObject_HEAD
 } srd_Decoder;
 
-/* This is only used for nicer srd_dbg() output. */
-static const char *OUTPUT_TYPES[] = {
-       "OUTPUT_ANN",
-       "OUTPUT_PYTHON",
-       "OUTPUT_BINARY",
-       "OUTPUT_META",
-};
+/* This is only used for nicer srd_dbg() output.
+ */
+static const char *output_type_name(unsigned int idx)
+{
+       static const char names[][16] = {
+               "OUTPUT_ANN",
+               "OUTPUT_PYTHON",
+               "OUTPUT_BINARY",
+               "OUTPUT_META",
+               "(invalid)"
+       };
+       return names[MIN(idx, G_N_ELEMENTS(names) - 1)];
+}
 
 static int convert_annotation(struct srd_decoder_inst *di, PyObject *obj,
                struct srd_proto_data *pdata)
@@ -45,8 +51,8 @@ static int convert_annotation(struct srd_decoder_inst *di, PyObject *obj,
 
        /* Should be a list of [annotation class, [string, ...]]. */
        if (!PyList_Check(obj) && !PyTuple_Check(obj)) {
-               srd_err("Protocol decoder %s submitted %s instead of list.",
-                       di->decoder->name, obj->ob_type->tp_name);
+               srd_err("Protocol decoder %s submitted an annotation that"
+                       " is not a list or tuple", di->decoder->name);
                return SRD_ERR_PYTHON;
        }
 
@@ -107,9 +113,8 @@ static int convert_binary(struct srd_decoder_inst *di, PyObject *obj,
 
        /* Should be a tuple of (binary class, bytes). */
        if (!PyTuple_Check(obj)) {
-               srd_err("Protocol decoder %s submitted SRD_OUTPUT_BINARY with "
-                               "%s instead of tuple.", di->decoder->name,
-                               obj->ob_type->tp_name);
+               srd_err("Protocol decoder %s submitted non-tuple for SRD_OUTPUT_BINARY.",
+                       di->decoder->name);
                return SRD_ERR_PYTHON;
        }
 
@@ -171,7 +176,7 @@ static int convert_meta(struct srd_proto_data *pdata, PyObject *obj)
        if (pdata->pdo->meta_type == G_VARIANT_TYPE_INT64) {
                if (!PyLong_Check(obj)) {
                        PyErr_Format(PyExc_TypeError, "This output was registered "
-                                       "as 'int', but '%s' was passed.", obj->ob_type->tp_name);
+                                       "as 'int', but something else was passed.");
                        return SRD_ERR_PYTHON;
                }
                intvalue = PyLong_AsLongLong(obj);
@@ -181,7 +186,7 @@ static int convert_meta(struct srd_proto_data *pdata, PyObject *obj)
        } else if (pdata->pdo->meta_type == G_VARIANT_TYPE_DOUBLE) {
                if (!PyFloat_Check(obj)) {
                        PyErr_Format(PyExc_TypeError, "This output was registered "
-                                       "as 'float', but '%s' was passed.", obj->ob_type->tp_name);
+                                       "as 'float', but something else was passed.");
                        return SRD_ERR_PYTHON;
                }
                dvalue = PyFloat_AsDouble(obj);
@@ -229,7 +234,7 @@ static PyObject *Decoder_put(PyObject *self, PyObject *args)
 
        srd_spew("Instance %s put %" PRIu64 "-%" PRIu64 " %s on oid %d.",
                 di->inst_id, start_sample, end_sample,
-                OUTPUT_TYPES[pdo->output_type], output_id);
+                output_type_name(pdo->output_type), output_id);
 
        pdata = g_malloc0(sizeof(struct srd_proto_data));
        pdata->start_sample = start_sample;
@@ -256,7 +261,7 @@ static PyObject *Decoder_put(PyObject *self, PyObject *args)
                        if (!(py_res = PyObject_CallMethod(
                                next_di->py_inst, "decode", "KKO", start_sample,
                                end_sample, py_data))) {
-                               srd_exception_catch("Calling %s decode(): ",
+                               srd_exception_catch("Calling %s decode() failed",
                                                        next_di->inst_id);
                        }
                        Py_XDECREF(py_res);
@@ -336,8 +341,7 @@ static PyObject *Decoder_register(PyObject *self, PyObject *args,
                else if (meta_type_py == &PyFloat_Type)
                        meta_type_gv = G_VARIANT_TYPE_DOUBLE;
                else {
-                       PyErr_Format(PyExc_TypeError, "Unsupported type '%s'.",
-                                       meta_type_py->tp_name);
+                       PyErr_Format(PyExc_TypeError, "Unsupported type.");
                        return NULL;
                }
        }
@@ -373,13 +377,24 @@ static PyMethodDef Decoder_methods[] = {
        {NULL, NULL, 0, NULL}
 };
 
-/** @cond PRIVATE */
-SRD_PRIV PyTypeObject srd_Decoder_type = {
-       PyVarObject_HEAD_INIT(NULL, 0)
-       .tp_name = "sigrokdecode.Decoder",
-       .tp_basicsize = sizeof(srd_Decoder),
-       .tp_flags = Py_TPFLAGS_DEFAULT | Py_TPFLAGS_BASETYPE,
-       .tp_doc = "sigrok Decoder base class",
-       .tp_methods = Decoder_methods,
-};
-/** @endcond */
+/** Create the sigrokdecode.Decoder type.
+ * @return The new type object.
+ * @private
+ */
+SRD_PRIV PyObject *srd_Decoder_type_new(void)
+{
+       PyType_Spec spec;
+       PyType_Slot slots[] = {
+               { Py_tp_doc, "sigrok Decoder base class" },
+               { Py_tp_methods, Decoder_methods },
+               { Py_tp_new, (void *)&PyType_GenericNew },
+               { 0, NULL }
+       };
+       spec.name = "sigrokdecode.Decoder";
+       spec.basicsize = sizeof(srd_Decoder);
+       spec.itemsize = 0;
+       spec.flags = Py_TPFLAGS_DEFAULT | Py_TPFLAGS_BASETYPE;
+       spec.slots = slots;
+
+       return PyType_FromSpec(&spec);
+}