[lvm-devel] master - cleanup: remove goto
Zdenek Kabelac
zkabelac at fedoraproject.org
Thu Nov 3 16:49:51 UTC 2016
Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=0d95082aa97b92d120daf1858acd91abe788d2cb
Commit: 0d95082aa97b92d120daf1858acd91abe788d2cb
Parent: 9cbe4c1af935bd4e76fbcc8a8736c6337bfc638b
Author: Zdenek Kabelac <zkabelac at redhat.com>
AuthorDate: Wed Oct 26 11:26:02 2016 +0200
Committer: Zdenek Kabelac <zkabelac at redhat.com>
CommitterDate: Thu Nov 3 17:49:07 2016 +0100
cleanup: remove goto
Move goto path into 'if()' branch.
---
daemons/dmeventd/plugins/mirror/dmeventd_mirror.c | 11 ++++-------
1 files changed, 4 insertions(+), 7 deletions(-)
diff --git a/daemons/dmeventd/plugins/mirror/dmeventd_mirror.c b/daemons/dmeventd/plugins/mirror/dmeventd_mirror.c
index 7653413..a7ed4dd 100644
--- a/daemons/dmeventd/plugins/mirror/dmeventd_mirror.c
+++ b/daemons/dmeventd/plugins/mirror/dmeventd_mirror.c
@@ -73,8 +73,10 @@ static int _get_mirror_event(struct dso_state *state, char *params)
unsigned i;
struct dm_status_mirror *ms;
- if (!dm_get_status_mirror(state->mem, params, &ms))
- goto_out;
+ if (!dm_get_status_mirror(state->mem, params, &ms)) {
+ log_error("Unable to parse mirror status string.");
+ return ME_IGNORE;
+ }
/* Check for bad mirror devices */
for (i = 0; i < ms->dev_count; ++i)
@@ -95,11 +97,6 @@ static int _get_mirror_event(struct dso_state *state, char *params)
dm_pool_free(state->mem, ms);
return r;
-
-out:
- log_error("Unable to parse mirror status string.");
-
- return ME_IGNORE;
}
static int _remove_failed_devices(const char *cmd_lvscan, const char *cmd_lvconvert)
More information about the lvm-devel
mailing list