aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--docs/man/nng_opts_parse.3supp.adoc2
-rw-r--r--src/supplemental/util/options.c4
-rw-r--r--src/supplemental/util/options.h2
-rw-r--r--tests/options.c119
-rw-r--r--tools/nngcat/nngcat.c8
5 files changed, 64 insertions, 71 deletions
diff --git a/docs/man/nng_opts_parse.3supp.adoc b/docs/man/nng_opts_parse.3supp.adoc
index 20a52109..2fbbe443 100644
--- a/docs/man/nng_opts_parse.3supp.adoc
+++ b/docs/man/nng_opts_parse.3supp.adoc
@@ -27,7 +27,7 @@ typedef struct nng_optspec {
bool o_arg; // Option takes an argument if true
} nng_optspec;
-int nng_opts_parse(int argc, char *const *argv, const nng_optspec *spec, int *val, const char **arg, int *idx);
+int nng_opts_parse(int argc, char *const *argv, const nng_optspec *spec, int *val, char **arg, int *idx);
----
== DESCRIPTION
diff --git a/src/supplemental/util/options.c b/src/supplemental/util/options.c
index c1881185..826a3692 100644
--- a/src/supplemental/util/options.c
+++ b/src/supplemental/util/options.c
@@ -17,13 +17,13 @@
// Call with optidx set to 1 to start parsing.
int
nng_opts_parse(int argc, char *const *argv, const nng_optspec *opts, int *val,
- const char **optarg, int *optidx)
+ char **optarg, int *optidx)
{
const nng_optspec *opt;
int matches;
bool shortopt;
size_t l;
- const char * arg = argv[*optidx];
+ char * arg = argv[*optidx];
int i;
if ((i = *optidx) >= argc) {
diff --git a/src/supplemental/util/options.h b/src/supplemental/util/options.h
index 392c4a53..83969a90 100644
--- a/src/supplemental/util/options.h
+++ b/src/supplemental/util/options.h
@@ -39,7 +39,7 @@ typedef struct nng_optspec nng_optspec;
// Returns -1 when the end of options is reached, 0 on success, or
// NNG_EINVAL if the option parse is invalid for any reason.
NNG_DECL int nng_opts_parse(int argc, char *const *argv,
- const nng_optspec *opts, int *val, const char **optarg, int *optidx);
+ const nng_optspec *opts, int *val, char **optarg, int *optidx);
#ifdef __cplusplus
}
diff --git a/tests/options.c b/tests/options.c
index db04eea6..2d81bd83 100644
--- a/tests/options.c
+++ b/tests/options.c
@@ -27,14 +27,12 @@ static nng_optspec case1[] = {
};
TestMain("Option Parsing", {
-
Convey("Simple works", {
-
- int opti = 1;
- const char *av[6];
- int ac = 5;
- int v;
- const char *a = NULL;
+ int opti = 1;
+ char *av[6];
+ int ac = 5;
+ int v;
+ char *a = NULL;
av[0] = "program";
av[1] = "-f";
@@ -55,12 +53,11 @@ TestMain("Option Parsing", {
});
Convey("Long works", {
-
- int opti = 1;
- const char *av[6];
- int ac = 5;
- int v;
- const char *a = NULL;
+ int opti = 1;
+ char *av[6];
+ int ac = 5;
+ int v;
+ char *a = NULL;
av[0] = "program";
av[1] = "--flag";
@@ -81,12 +78,11 @@ TestMain("Option Parsing", {
});
Convey("Attached short works", {
-
- int opti = 1;
- const char *av[3];
- int ac = 3;
- int v;
- const char *a = NULL;
+ int opti = 1;
+ char *av[3];
+ int ac = 3;
+ int v;
+ char *a = NULL;
av[0] = "program";
av[1] = "-v123";
@@ -101,12 +97,11 @@ TestMain("Option Parsing", {
});
Convey("Attached long (=) works", {
-
- int opti = 1;
- const char *av[3];
- int ac = 3;
- int v;
- const char *a = NULL;
+ int opti = 1;
+ char *av[3];
+ int ac = 3;
+ int v;
+ char *a = NULL;
av[0] = "program";
av[1] = "--value=123";
@@ -121,12 +116,11 @@ TestMain("Option Parsing", {
});
Convey("Attached long (:) works", {
-
- int opti = 1;
- const char *av[3];
- int ac = 3;
- int v;
- const char *a = NULL;
+ int opti = 1;
+ char *av[3];
+ int ac = 3;
+ int v;
+ char *a = NULL;
av[0] = "program";
av[1] = "--value:123";
@@ -141,12 +135,11 @@ TestMain("Option Parsing", {
});
Convey("Negative bad short works", {
-
- int opti = 1;
- const char *av[3];
- int ac = 3;
- int v;
- const char *a = NULL;
+ int opti = 1;
+ char *av[3];
+ int ac = 3;
+ int v;
+ char *a = NULL;
av[0] = "program";
av[1] = "-Z";
@@ -156,12 +149,11 @@ TestMain("Option Parsing", {
});
Convey("Negative bad long works", {
-
- int opti = 1;
- const char *av[3];
- int ac = 3;
- int v;
- const char *a = NULL;
+ int opti = 1;
+ char *av[3];
+ int ac = 3;
+ int v;
+ char *a = NULL;
av[0] = "program";
av[1] = "--something";
@@ -171,11 +163,11 @@ TestMain("Option Parsing", {
});
Convey("Separator flag works", {
- int opti = 1;
- const char *av[5];
- int ac = 5;
- int v;
- const char *a = NULL;
+ int opti = 1;
+ char *av[5];
+ int ac = 5;
+ int v;
+ char *a = NULL;
av[0] = "program";
av[1] = "-f";
@@ -190,22 +182,22 @@ TestMain("Option Parsing", {
});
Convey("No options works", {
- int opti = 1;
- const char *av[1];
- int ac = 1;
- int v;
- const char *a = NULL;
+ int opti = 1;
+ char *av[1];
+ int ac = 1;
+ int v;
+ char *a = NULL;
av[0] = "program";
So(nng_opts_parse(ac, av, case1, &v, &a, &opti) == -1);
});
Convey("No options (but arguments) works", {
- int opti = 1;
- const char *av[2];
- int ac = 2;
- int v;
- const char *a = NULL;
+ int opti = 1;
+ char *av[2];
+ int ac = 2;
+ int v;
+ char *a = NULL;
av[0] = "program";
av[1] = "123";
@@ -213,12 +205,11 @@ TestMain("Option Parsing", {
So(opti == 1);
});
Convey("Mixed long and short works", {
-
- int opti = 1;
- const char *av[7];
- int ac = 7;
- int v;
- const char *a = NULL;
+ int opti = 1;
+ char *av[7];
+ int ac = 7;
+ int v;
+ char *a = NULL;
av[0] = "program";
av[1] = "--value=123";
diff --git a/tools/nngcat/nngcat.c b/tools/nngcat/nngcat.c
index 132599e4..e81f59c0 100644
--- a/tools/nngcat/nngcat.c
+++ b/tools/nngcat/nngcat.c
@@ -200,7 +200,9 @@ static nng_optspec opts[] = {
.o_arg = true,
},
{
- .o_name = "zt-home", .o_val = OPT_ZTHOME, .o_arg = true,
+ .o_name = "zt-home",
+ .o_val = OPT_ZTHOME,
+ .o_arg = true,
},
{ .o_name = "version", .o_short = 'V', .o_val = OPT_VERSION },
@@ -671,10 +673,10 @@ sendrecv(nng_socket sock)
}
int
-main(int ac, const char **av)
+main(int ac, char **av)
{
int idx;
- const char * arg;
+ char * arg;
int val;
int rv;
char scratch[512];