Re: [PATCH 2/2] Staging: Lustre: Fixed coding style issues

[Joe Perches]

On Sun, 2016-04-17 at 21:35 +0000, Drokin, Oleg wrote:
> On Apr 17, 2016, at 10:11 AM, Panos Vlachos wrote:
> > Fixed one coding style issue in the
> > file router_proc.c (Lustre staging driver)
> Perhaps it's best to reference what the issue is.
> I.e. "Fix too long line in â"
> and then in the actual message add that you just split it?
[]
> > diff --git a/drivers/staging/lustre/lnet/lnet/router_proc.c b/drivers/staging/lustre/lnet/lnet/router_proc.c
[]
> > @@ -32,7 +32,8 @@
> > Â*/
> > #define LNET_PROC_CPT_BITS	(LNET_CPT_BITS + 1)
> > /* change version, 16 bits or 8 bits */
> > -#define LNET_PROC_VER_BITS	max_t(size_t, min_t(size_t, LNET_LOFFT_BITS, 64) / 4, 8)
> > +#define LNET_PROC_VER_BITS	\
> > +max_t(size_t, min_t(size_t, LNET_LOFFT_BITS, 64) / 4, 8)
> Hm, I am not sure this is an improvement, you might want to at least place a tab at the start of this
> second line, so that indentation helps us better see it's a continuation of the previous one.

Given:
	#define LNET_LOFFT_BITSÂÂÂÂÂÂÂÂÂÂ(sizeof(loff_t) * 8)
and:
	include/linux/types.h:typedef __kernel_loff_tÂÂÂÂÂÂÂÂÂÂÂloff_t;
	include/uapi/asm-generic/posix_types.h:typedef long longÂÂÂÂÂÂÂÂ__kernel_loff_t;

and this isn't going to change any time soon,
maybe this #define isn't particularly useful and

	#define LNET_PROC_VER_BITS	16

is sufficient.