[PATCH] staging: lustre: Fix coding style. Switch and case at the same indent.

Greg KH gregkh at linuxfoundation.org
Wed Oct 29 08:38:25 UTC 2014


On Tue, Oct 28, 2014 at 09:35:15PM +0200, Evaldas Palaima wrote:
> Fix a style issue reported by checkpatch.pl for the Eudyptula challenge.
> 
> Signed-off-by: Evaldas Palaima <evaldas.palaima at gmail.com>
> ---
>  drivers/staging/lustre/lustre/libcfs/fail.c | 24 ++++++++++++------------
>  1 file changed, 12 insertions(+), 12 deletions(-)
> 
> diff --git a/drivers/staging/lustre/lustre/libcfs/fail.c b/drivers/staging/lustre/lustre/libcfs/fail.c
> index e73ca3d..92444b0 100644
> --- a/drivers/staging/lustre/lustre/libcfs/fail.c
> +++ b/drivers/staging/lustre/lustre/libcfs/fail.c
> @@ -103,18 +103,18 @@ int __cfs_fail_check_set(__u32 id, __u32 value, int set)
>  	}
>  
>  	switch (set) {
> -		case CFS_FAIL_LOC_NOSET:
> -		case CFS_FAIL_LOC_VALUE:
> -			break;
> -		case CFS_FAIL_LOC_ORSET:
> -			cfs_fail_loc |= value & ~(CFS_FAILED | CFS_FAIL_ONCE);
> -			break;
> -		case CFS_FAIL_LOC_RESET:
> -			cfs_fail_loc = value;
> -			break;
> -		default:
> -			LASSERTF(0, "called with bad set %u\n", set);
> -			break;
> +	case CFS_FAIL_LOC_NOSET:
> +	case CFS_FAIL_LOC_VALUE:
> +		break;
> +	case CFS_FAIL_LOC_ORSET:
> +		cfs_fail_loc |= value & ~(CFS_FAILED | CFS_FAIL_ONCE);
> +		break;
> +	case CFS_FAIL_LOC_RESET:
> +		cfs_fail_loc = value;
> +		break;
> +	default:
> +		LASSERTF(0, "called with bad set %u\n", set);
> +		break;
>  	}
>  
>  	return 1;

Someone else did this change before you, sorry :(


More information about the devel mailing list