Re: [PATCH] staging: android: binder: Fix 10 overlong lines

From: Greg KH
Date: Thu Jun 19 2014 - 18:43:21 EST


On Sat, May 31, 2014 at 05:29:53PM -0600, Daniel Dressler wrote:
> This patch was made following Greg Kroah-Hartman's
> newbie guide. It is not intended to make any
> logical alterations to codeflow. Of note to reviews
> this patch switch's a call to kzalloc to kcalloc on
> line 2837
>
> Neil Brown gave good advice on how to align
> the over-long function calls. He suggested
> using a single tab to indent the arguments.
> Like thus:
> var = function(
> arg1, arg2, arg3, arg4);
>
> I intended to use the style he suggested but
> noticed much of the existing code instead
> aligns the overflow with the first argument
> and tries to give all arguments equal space.
> Like thus:
> var = function(arg1, arg2
> arg3, arg4);
>
> This second method uses spaces to align the
> arguments.
>
> Taking a survey of the first 800 lines the
> following lines used each method:
>
> Method #1: 666, 527, 572
> Method #2: 475, 495, 520, 545, 554, 582, 630,
> 677, 788
>
> Thus to maintain consistancy this patch uses
> method #1. If method #1 is not acceptable
> please do not hesitate to say so.
>
> Signed-off-by: Daniel Dressler <danieru.dressler@xxxxxxxxx>
> ---
> drivers/staging/android/binder.c | 32 ++++++++++++++++++++++----------
> 1 file changed, 22 insertions(+), 10 deletions(-)

This patch doesn't apply properly to my staging-next branch of the
staging.git tree :(

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/