[OpenWrt-Devel] [PATCH fstools 2/4] block: move blockd_notify() call out of the conditional blocks

[Rafał Miłecki]

From: Rafał Miłecki <rafal at milecki.pl>

That function is called no matter if "m" variable is null or not. This
saves a one duplicated line & allows merging two conditional blocks
starting with if (m). It doesn't change code execution order.

Signed-off-by: Rafał Miłecki <rafal at milecki.pl>
---
 block.c | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/block.c b/block.c
index d87251d..f42d0d6 100644
--- a/block.c
+++ b/block.c
@@ -1021,10 +1021,16 @@ static int mount_device(struct probe_info *pr, int type)
 	if (m && m->extroot)
 		return -1;
 
+	if (type == TYPE_HOTPLUG)
+		blockd_notify(device, m, pr);
+
 	if (m) {
+		char *target = m->target;
+		char _target[32];
+		int err = 0;
+
 		switch (type) {
 		case TYPE_HOTPLUG:
-			blockd_notify(device, m, pr);
 			if (m->autofs)
 				return 0;
 			if (!auto_mount)
@@ -1039,14 +1045,6 @@ static int mount_device(struct probe_info *pr, int type)
 				return -1;
 			break;
 		}
-	} else if (type == TYPE_HOTPLUG) {
-		blockd_notify(device, NULL, pr);
-	}
-
-	if (m) {
-		char *target = m->target;
-		char _target[32];
-		int err = 0;
 
 		if (m->autofs) {
 			snprintf(_target, sizeof(_target), "/tmp/run/blockd/%s", device);
-- 
2.13.7


_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel