diff options
| author | Kaleb S. KEITHLEY <kkeithle@redhat.com> | 2016-09-13 05:53:14 -0400 |
|---|---|---|
| committer | Kaleb KEITHLEY <kkeithle@redhat.com> | 2016-09-13 21:08:36 -0700 |
| commit | 0419ab2820820f1ac93e55dbf3a665824560bade (patch) | |
| tree | d1eb531dd9d3845814dfc1f608b10cbda8a5aa27 | |
| parent | 71d3c5850d33a60115e561f88072826fd81bc1a0 (diff) | |
xlators/{dht,tier}: 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.
BUG: 1369124
Change-Id: I6feee863927254ae9f59013112bcc297ad09e89b
Signed-off-by: Kaleb S. KEITHLEY <kkeithle@redhat.com>
Reviewed-on: http://review.gluster.org/15477
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Dan Lambright <dlambrig@redhat.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: N Balachandran <nbalacha@redhat.com>
| -rw-r--r-- | xlators/cluster/dht/src/dht-common.c | 2 | ||||
| -rw-r--r-- | xlators/cluster/dht/src/tier.c | 4 |
2 files changed, 0 insertions, 6 deletions
diff --git a/xlators/cluster/dht/src/dht-common.c b/xlators/cluster/dht/src/dht-common.c index 65f4f3d2ef7..b984db951c1 100644 --- a/xlators/cluster/dht/src/dht-common.c +++ b/xlators/cluster/dht/src/dht-common.c @@ -8738,8 +8738,6 @@ dht_ipc_cbk (call_frame_t *frame, void *cookie, xlator_t *this, { dht_local_t *local = NULL; int this_call_cnt = 0; - dht_layout_t *layout = NULL; - int ret = -1; GF_VALIDATE_OR_GOTO ("dht", frame, out); GF_VALIDATE_OR_GOTO ("dht", this, out); diff --git a/xlators/cluster/dht/src/tier.c b/xlators/cluster/dht/src/tier.c index f54d4f6cdc6..2f8eddbc07d 100644 --- a/xlators/cluster/dht/src/tier.c +++ b/xlators/cluster/dht/src/tier.c @@ -1700,7 +1700,6 @@ tier_process_self_compact (tier_brick_list_t *local_brick, void *args) dict_t *params_dict = NULL; dict_t *ctr_ipc_dict = NULL; gfdb_brick_info_t *gfdb_brick_info = args; - int is_changing = -1; /*Init of all the essentials*/ GF_VALIDATE_OR_GOTO ("tier", gfdb_brick_info , out); @@ -1826,7 +1825,6 @@ static int tier_compact_db_brick (tier_brick_list_t *local_brick, void *args) { int ret = -1; - char *strval = NULL; GF_VALIDATE_OR_GOTO ("tier", local_brick, out); @@ -2232,9 +2230,7 @@ static void int check_watermark = 0; gf_defrag_info_t *defrag = NULL; xlator_t *this = NULL; - struct list_head *bricklist_temp = NULL; migration_args_t *args = in_args; - gf_boolean_t compacted = _gf_false; GF_VALIDATE_OR_GOTO ("tier", args, out); GF_VALIDATE_OR_GOTO ("tier", args->brick_list, out); |