From abb4cbeea35c40d69f18aba599f58d7e2dc5fdaf Mon Sep 17 00:00:00 2001 From: Jeff Darcy Date: Thu, 8 Sep 2011 11:07:10 -0400 Subject: Second round of warning suppression. Used a #pragma to kill ~170 in rpcgen code. Added GF_UNUSED to deal with a few more from macros elsewhere. The remainder are function return values (mostly context and dict calls) that really should be checked. Those would be harder to fix without real understanding of the code where they occur, so they remain as reminders. (Patchset 2: deal with older gcc that doesn't handle #pragma GCC diagnostic) (Patchset 3: fix include paths in generated files) (Patchset 4: keep up with trunk, squash 9 new warnings) (Patchset 5: six more, all in AFR) Change-Id: I29760c8c81be4d7e6489312c5d0e92cc24814b7b BUG: 2550 Reviewed-on: http://review.gluster.com/378 Tested-by: Gluster Build System Reviewed-by: Vijay Bellur --- xlators/cluster/afr/src/pump.c | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) (limited to 'xlators/cluster/afr/src/pump.c') diff --git a/xlators/cluster/afr/src/pump.c b/xlators/cluster/afr/src/pump.c index 63c89b3c7b2..dbf86b0a2a6 100644 --- a/xlators/cluster/afr/src/pump.c +++ b/xlators/cluster/afr/src/pump.c @@ -249,13 +249,11 @@ static int pump_update_resume_state (xlator_t *this, const char *path) { afr_private_t *priv = NULL; - pump_private_t *pump_priv = NULL; pump_state_t state; const char *resume_path = NULL; priv = this->private; - pump_priv = priv->pump_private; state = pump_get_state (); @@ -284,14 +282,10 @@ pump_update_resume_state (xlator_t *this, const char *path) static gf_boolean_t is_pump_traversal_allowed (xlator_t *this, const char *path) { - afr_private_t *priv = NULL; - pump_state_t state; const char *resume_path = NULL; gf_boolean_t ret = _gf_true; - priv = this->private; - state = pump_get_state (); if (state == PUMP_STATE_RESUME) { @@ -472,12 +466,8 @@ out: static int pump_update_resume_path (xlator_t *this) { - afr_private_t *priv = NULL; - const char *resume_path = NULL; - priv = this->private; - resume_path = pump_get_resume_path (this); if (resume_path) { @@ -2386,7 +2376,7 @@ init (xlator_t *this) xlator_list_t * trav = NULL; int i = 0; int ret = -1; - int op_errno = 0; + GF_UNUSED int op_errno = 0; int source_child = 0; -- cgit