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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxx>
> ---
>  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 :(
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux