Various #include file cosmetic fixes.

[sigrok-cli.git] / main.c

diff --git a/main.c b/main.c
index 1d3b33df800f1765f6247a7acafe33c603dd94c0..a53566e8960b31b24ce3b89e6e68edd5b62e841d 100644 (file)
--- a/main.c
+++ b/main.c
@@ -17,9 +17,9 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include "sigrok-cli.h"
 #include <stdlib.h>
 #include <glib.h>
+#include "sigrok-cli.h"
 
 struct sr_context *sr_ctx = NULL;
 #ifdef HAVE_SRD
@@ -50,6 +50,7 @@ static void logger(const gchar *log_domain, GLogLevelFlags log_level,
 int select_channels(struct sr_dev_inst *sdi)
 {
        struct sr_channel *ch;
+       gboolean enabled;
        GSList *selected_channels, *l, *channels;
 
        channels = sr_dev_inst_channels_get(sdi);
@@ -60,10 +61,9 @@ int select_channels(struct sr_dev_inst *sdi)
 
                for (l = channels; l; l = l->next) {
                        ch = l->data;
-                       if (g_slist_find(selected_channels, ch))
-                               ch->enabled = TRUE;
-                       else
-                               ch->enabled = FALSE;
+                       enabled = (g_slist_find(selected_channels, ch) != NULL);
+                       if (sr_dev_channel_enable(ch, enabled) != SR_OK)
+                               return SR_ERR;
                }
                g_slist_free(selected_channels);
        }
@@ -80,22 +80,26 @@ gboolean config_key_has_cap(struct sr_dev_driver *driver,
        GVariant *gvar_opts;
        const uint32_t *opts;
        gsize num_opts, i;
+       gboolean result;
 
        if (sr_config_list(driver, sdi, cg, SR_CONF_DEVICE_OPTIONS,
                        &gvar_opts) != SR_OK)
                return FALSE;
 
        opts = g_variant_get_fixed_array(gvar_opts, &num_opts, sizeof(uint32_t));
+       result = FALSE;
        for (i = 0; i < num_opts; i++) {
                if ((opts[i] & SR_CONF_MASK) == key) {
                        if ((opts[i] & capability) == capability)
-                               return TRUE;
+                               result = TRUE;
                        else
-                               return FALSE;
+                               result = FALSE;
+                       break;
                }
        }
+       g_variant_unref(gvar_opts);
 
-       return FALSE;
+       return result;
 }
 
 int maybe_config_get(struct sr_dev_driver *driver,