diff options
author | Niels de Vos <ndevos@redhat.com> | 2016-09-18 08:23:23 +0200 |
---|---|---|
committer | Niels de Vos <ndevos@redhat.com> | 2016-09-18 09:34:18 -0700 |
commit | 09ce988ad6d8e0576f1fc23abf71b43ac4b7514e (patch) | |
tree | 42920a2f6e3f6a4ee85d12c75aac2f607a6c86a4 /cli/src | |
parent | ee1e0994994314453897d97fb28d91d50c40f40e (diff) |
cli: fix unused variable warnings/errors
http://review.gluster.org/14085 fixes a "pragma leak" where the
generated rpc/xdr headers have a pair of pragmas that disable these
warnings. With the warnings disabled, many unused variables have
crept into the code base.
And 14085 won't pass its own smoke test until all these warnings are
fixed.
Change-Id: Ib78fed1fab00ba22a90f2148a343170488fa05cf
BUG: 1369124
Signed-off-by: Niels de Vos <ndevos@redhat.com>
Reviewed-on: http://review.gluster.org/15523
Smoke: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Diffstat (limited to 'cli/src')
-rw-r--r-- | cli/src/cli-cmd-parser.c | 1 | ||||
-rw-r--r-- | cli/src/cli-rpc-ops.c | 2 |
2 files changed, 0 insertions, 3 deletions
diff --git a/cli/src/cli-cmd-parser.c b/cli/src/cli-cmd-parser.c index 86b02877159..d6ac6fd7077 100644 --- a/cli/src/cli-cmd-parser.c +++ b/cli/src/cli-cmd-parser.c @@ -930,7 +930,6 @@ cli_cmd_get_state_parse (struct cli_state *state, { dict_t *dict = NULL; int ret = -1; - uint32_t cmd = 0; char *odir = NULL; char *filename = NULL; char *daemon_name = NULL; diff --git a/cli/src/cli-rpc-ops.c b/cli/src/cli-rpc-ops.c index f20e962a61e..f217e13abdd 100644 --- a/cli/src/cli-rpc-ops.c +++ b/cli/src/cli-rpc-ops.c @@ -4334,8 +4334,6 @@ gf_cli_get_state (call_frame_t *frame, xlator_t *this, void *data) int ret = 0; dict_t *dict = NULL; - char *odir = NULL; - if (!frame || !this || !data) { ret = -1; goto out; |