diff options
author | Maciej Żenczykowski <maze@google.com> | 2019-10-17 11:20:51 -0700 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2019-10-29 13:51:21 -0400 |
commit | 2054a8c77006e98957b7451ad4f60c2b22186cd0 (patch) | |
tree | 992d8a12cd64188f8e24972e70de5a490391a078 | |
parent | d5432a966f13a90a0cd429d3f4d8785905be92e9 (diff) | |
download | ethtool-2054a8c77006e98957b7451ad4f60c2b22186cd0.tar.gz |
fix unused parameter warning in find_option()
This fixes:
external/ethtool/ethtool.c:5417:28: error: unused parameter 'argc' [-Werror,-Wunused-parameter]
static int find_option(int argc, char **argp)
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: I7782cac5f58e217fc6032037a2f248b03573b9d4
Signed-off-by: John W. Linville <linville@tuxdriver.com>
-rw-r--r-- | ethtool.c | 9 |
1 files changed, 4 insertions, 5 deletions
@@ -5507,7 +5507,7 @@ static int show_usage(struct cmd_context *ctx maybe_unused) return 0; } -static int find_option(int argc, char **argp) +static int find_option(char *arg) { const char *opt; size_t len; @@ -5517,8 +5517,7 @@ static int find_option(int argc, char **argp) opt = args[k].opts; for (;;) { len = strcspn(opt, "|"); - if (strncmp(*argp, opt, len) == 0 && - (*argp)[len] == 0) + if (strncmp(arg, opt, len) == 0 && arg[len] == 0) return k; if (opt[len] == 0) @@ -5667,7 +5666,7 @@ static int do_perqueue(struct cmd_context *ctx) ctx->argp++; } - i = find_option(ctx->argc, ctx->argp); + i = find_option(ctx->argp[0]); if (i < 0) exit_bad_args(); @@ -5719,7 +5718,7 @@ int main(int argc, char **argp) if (argc == 0) exit_bad_args(); - k = find_option(argc, argp); + k = find_option(*argp); if (k >= 0) { argp++; argc--; |