From: Rafał Miłecki <[email protected]> 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 <[email protected]> --- 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 [email protected] https://lists.openwrt.org/mailman/listinfo/openwrt-devel
