diff options
author | Barak Sason <bsasonro@redhat.com> | 2019-08-18 15:54:11 +0300 |
---|---|---|
committer | Ravishankar N <ravishankar@redhat.com> | 2019-08-24 04:28:00 +0000 |
commit | 7afea5eeef61d3f29496b15ba54defeb39161588 (patch) | |
tree | 276ba5b7ef05d240a0244694c0b46fcb0e9224ff /xlators/cluster/afr/src/afr-self-heal-common.c | |
parent | 673f2f90f35e5ea381bd22a43403278f89c957a5 (diff) |
cluster/afr - Unused variables
-Minor change to if-else structure to avoid code duplication.
-Added logging in case method calls fails
CID: 1394654
Updates: bz#789278
Change-Id: Ibef4450dc89ddd3bf951303d5b87f503924fd250
Signed-off-by: Barak Sason <bsasonro@redhat.com>
Diffstat (limited to 'xlators/cluster/afr/src/afr-self-heal-common.c')
-rw-r--r-- | xlators/cluster/afr/src/afr-self-heal-common.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/xlators/cluster/afr/src/afr-self-heal-common.c b/xlators/cluster/afr/src/afr-self-heal-common.c index 4dc38e14099..b62b4f8ebaa 100644 --- a/xlators/cluster/afr/src/afr-self-heal-common.c +++ b/xlators/cluster/afr/src/afr-self-heal-common.c @@ -1772,11 +1772,9 @@ afr_selfheal_discover_cbk(call_frame_t *frame, void *cookie, xlator_t *this, if (xdata) { local->replies[i].xdata = dict_ref(xdata); ret = dict_get_int8(xdata, "link-count", &need_heal); - local->replies[i].need_heal = need_heal; - } else { - local->replies[i].need_heal = need_heal; } + local->replies[i].need_heal = need_heal; syncbarrier_wake(&local->barrier); return 0; |