Add --get option.

[sigrok-cli.git] / options.c

diff --git a/options.c b/options.c
index 9f9d912fcd3f5c0c399017dab57750ebb951d2ce..179d892f4021239bda222ee4fdd84e9462e3c7b3 100644 (file)
--- a/options.c
+++ b/options.c
@@ -45,6 +45,7 @@ gchar *opt_time = NULL;
 gchar *opt_samples = NULL;
 gchar *opt_frames = NULL;
 gchar *opt_continuous = NULL;
+gchar *opt_get = NULL;
 gchar *opt_set = NULL;
 
 /* defines a callback function that generates
@@ -85,9 +86,13 @@ CHECK_ONCE(opt_pd_binary)
 CHECK_ONCE(opt_time)
 CHECK_ONCE(opt_samples)
 CHECK_ONCE(opt_frames)
+CHECK_ONCE(opt_get)
 
 #undef CHECK_STR_ONCE
 
+static gchar **input_file_array = NULL;
+static gchar **output_file_array = NULL;
+
 static const GOptionEntry optargs[] = {
        {"version", 'V', 0, G_OPTION_ARG_NONE, &opt_version,
                        "Show version and support list", NULL},
@@ -97,11 +102,11 @@ static const GOptionEntry optargs[] = {
                        "The driver to use", NULL},
        {"config", 'c', 0, G_OPTION_ARG_CALLBACK, &check_opt_config,
                        "Specify device configuration options", NULL},
-       {"input-file", 'i', 0, G_OPTION_ARG_FILENAME, &opt_input_file,
+       {"input-file", 'i', 0, G_OPTION_ARG_FILENAME_ARRAY, &input_file_array,
                        "Load input from file", NULL},
        {"input-format", 'I', 0, G_OPTION_ARG_CALLBACK, &check_opt_input_format,
                        "Input format", NULL},
-       {"output-file", 'o', 0, G_OPTION_ARG_FILENAME, &opt_output_file,
+       {"output-file", 'o', 0, G_OPTION_ARG_FILENAME_ARRAY, &output_file_array,
                        "Save output to file", NULL},
        {"output-format", 'O', 0, G_OPTION_ARG_CALLBACK, &check_opt_output_format,
                        "Output format", NULL},
@@ -137,6 +142,7 @@ static const GOptionEntry optargs[] = {
                        "Number of frames to acquire", NULL},
        {"continuous", 0, 0, G_OPTION_ARG_NONE, &opt_continuous,
                        "Sample continuously", NULL},
+       {"get", 0, 0, G_OPTION_ARG_CALLBACK, &check_opt_get, "Get device option only", NULL},
        {"set", 0, 0, G_OPTION_ARG_NONE, &opt_set, "Set device options only", NULL},
        {NULL, 0, 0, 0, NULL, NULL, NULL}
 };
@@ -156,10 +162,41 @@ int parse_options(int argc, char **argv)
                goto done;
        }
 
+       /*
+        * Because of encoding issues with filenames (mentioned in the glib
+        * documentation), we don't check them with a callback function, but
+        * collect them into arrays and then check if the arrays contain at
+        * most one element.
+        */
+       if (NULL != input_file_array) {
+               if (NULL != input_file_array[0] && NULL != input_file_array[1]) {
+                       g_critical("option \"--input-file/-i\" only allowed once");
+                       goto done;
+               }
+               opt_input_file = g_strdup(input_file_array[0]);
+       }
+
+       if (NULL != output_file_array) {
+               if (NULL != output_file_array[0] && NULL != output_file_array[1]) {
+                       g_critical("option \"--output-file/-o\" only allowed once");
+                       goto done;
+               }
+               opt_output_file = g_strdup(output_file_array[0]);
+       }
+
+       if (1 != argc) {
+               g_critical("superfluous command line argument \"%s\"", argv[1]);
+               goto done;
+       }
+
        ret = 0;
 
 done:
        g_option_context_free(context);
+       g_strfreev(input_file_array);
+       g_strfreev(output_file_array);
+       input_file_array = NULL;
+       output_file_array = NULL;
 
        return ret;
 }