Use consistent __init__() format across all PDs.

[libsigrokdecode.git] / decoder.c

diff --git a/decoder.c b/decoder.c
index 5fce5a00026125b8575b0776f26badd6a700ef0e..748280af68b30fd6460e4df32135bf467c27a232 100644 (file)
--- a/decoder.c
+++ b/decoder.c
@@ -39,7 +39,7 @@
 
 /** @cond PRIVATE */
 
-/* The list of protocol decoders. */
+/* The list of loaded protocol decoders. */
 static GSList *pd_list = NULL;
 
 /* srd.c */
@@ -64,7 +64,7 @@ static gboolean srd_check_init(void)
 }
 
 /**
- * Returns the list of supported/loaded protocol decoders.
+ * Returns the list of loaded protocol decoders.
  *
  * This is a GSList of pointers to struct srd_decoder items.
  *
@@ -597,7 +597,7 @@ static int check_method(PyObject *py_dec, const char *mod_name,
  */
 SRD_API int srd_decoder_load(const char *module_name)
 {
-       PyObject *py_modname, *py_basedec, *py_apiver;
+       PyObject *py_basedec, *py_apiver;
        struct srd_decoder *d;
        long apiver;
        int is_subclass;
@@ -617,13 +617,7 @@ SRD_API int srd_decoder_load(const char *module_name)
 
        d = g_malloc0(sizeof(struct srd_decoder));
 
-       /* Import the Python module. */
-       py_modname = PyUnicode_FromString(module_name);
-       if (!py_modname)
-               goto except_out;
-
-       d->py_mod = PyImport_Import(py_modname);
-       Py_DECREF(py_modname);
+       d->py_mod = py_import_by_name(module_name);
        if (!d->py_mod)
                goto except_out;
 
@@ -710,7 +704,7 @@ SRD_API int srd_decoder_load(const char *module_name)
        if (get_binary_classes(d) != SRD_OK)
                goto err_out;
 
-       /* Append it to the list of supported/loaded decoders. */
+       /* Append it to the list of loaded decoders. */
        pd_list = g_slist_append(pd_list, d);
 
        return SRD_OK;
@@ -793,6 +787,9 @@ SRD_API int srd_decoder_unload(struct srd_decoder *dec)
                srd_inst_free_all(sess, NULL);
        }
 
+       /* Remove the PD from the list of loaded decoders. */
+       pd_list = g_slist_remove(pd_list, dec);
+
        decoder_free(dec);
 
        return SRD_OK;
@@ -806,15 +803,9 @@ static void srd_decoder_load_all_zip_path(char *path)
        char *prefix;
        size_t prefix_len;
 
-       zipimport_mod = NULL;
        set = files = prefix_obj = zipimporter = zipimporter_class = NULL;
 
-       modname = PyUnicode_FromString("zipimport");
-       if (!modname)
-               goto err_out;
-
-       zipimport_mod = PyImport_Import(modname);
-       Py_DECREF(modname);
+       zipimport_mod = py_import_by_name("zipimport");
        if (zipimport_mod == NULL)
                goto err_out;