From 0419ab2820820f1ac93e55dbf3a665824560bade Mon Sep 17 00:00:00 2001 From: "Kaleb S. KEITHLEY" Date: Tue, 13 Sep 2016 05:53:14 -0400 Subject: 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 Reviewed-on: http://review.gluster.org/15477 Smoke: Gluster Build System NetBSD-regression: NetBSD Build System Reviewed-by: Dan Lambright CentOS-regression: Gluster Build System Reviewed-by: N Balachandran --- xlators/cluster/dht/src/dht-common.c | 2 -- xlators/cluster/dht/src/tier.c | 4 ---- 2 files changed, 6 deletions(-) (limited to 'xlators/cluster') 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); -- cgit