[OpenWrt-Devel] [PATCH fstools 1/4] block: fix formatting & indent in the mount_device()

Michael Heimpold mhei at heimpold.de
Sat Dec 1 07:32:02 EST 2018


Hi,

Am Freitag, 30. November 2018, 14:09:20 CET schrieb Rafał Miłecki:
> From: Rafał Miłecki <rafal at milecki.pl>
> 
> There was some misformatted code that seemed like switch block having an
> else block. This improves code readability.
> 
> Signed-off-by: Rafał Miłecki <rafal at milecki.pl>
> ---
>  block.c | 34 ++++++++++++++++++----------------
>  1 file changed, 18 insertions(+), 16 deletions(-)
> 
> diff --git a/block.c b/block.c
> index b01a633..d87251d 100644
> --- a/block.c
> +++ b/block.c
> @@ -1021,22 +1021,24 @@ static int mount_device(struct probe_info *pr, int
> type) if (m && m->extroot)
>  		return -1;
> 
> -	if (m) switch (type) {
> -	case TYPE_HOTPLUG:
> -		blockd_notify(device, m, pr);
> -		if (m->autofs)
> -			return 0;
> -		if (!auto_mount)
> -			return -1;
> -		break;
> -	case TYPE_AUTOFS:
> -		if (!m->autofs)
> -			return -1;
> -		break;
> -	case TYPE_DEV:
> -		if (m->autofs)
> -			return -1;
> -		break;
> +	if (m) {
> +		switch (type) {
> +		case TYPE_HOTPLUG:
> +			blockd_notify(device, m, pr);
> +			if (m->autofs)
> +				return 0;
> +			if (!auto_mount)
> +				return -1;
> +			break;
> +		case TYPE_AUTOFS:
> +			if (!m->autofs)
> +				return -1;
> +			break;
> +		case TYPE_DEV:
> +			if (m->autofs)
> +				return -1;
> +			break;
> +		}
>  	} else if (type == TYPE_HOTPLUG) {
>  		blockd_notify(device, NULL, pr);
>  	}

LGTM, so feel free to add for the whole series:

Reviewed-by: Michael Heimpold <mhei at heimpold.de>




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


More information about the openwrt-devel mailing list