Re: [PATCH] Staging: android: binder: Fix over-80-char lines

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

 



Thank you Neil

I was not sure if space-based alignment was allowed so thank you for
demonstrating the change. It does make the code much easier to read.

Similar thank you for how to indent over-long function calls. I had
mis-understood the coding standard to mean overflow needed to be on
the far right margin. A single indent before the first argument looks
much nicer and consistent.

Thank you Dan

You are right! It does break the code. Thank you for finding that and
the name issue. I'll rewrite the patch on top of linux-next this time
with the proper style as Neil suggested. I think git send-email added
the danieru. I had tried fixing the name my git config and then git
amending but send-email continued adding the danieru address so I
assumed that it always took the current user's username. Now it sounds
like that was some metadata git did not redo upon amend.

Daniel

2014-05-31 0:51 GMT-06:00 Dan Carpenter <dan.carpenter@xxxxxxxxxx>:
> What Neil said.
>
> On Fri, May 30, 2014 at 10:31:20PM -0600, Daniel Dressler wrote:
>> From: danieru <danieru.dressler@xxxxxxxxx>
>
> This is wrong and we also get your name and email address from the
> email headers so leave this line out.
>
>>
>> Following Greg Kroah-Hartman's newbie guide to hacking
>> the linux kernel this patch addresses only coding style
>> issues.
>>
>> Binder still has many too-long lines but I'm worried
>> doing too much work in a single patch is unfair to the
>> reviewers. So this patch address 20% of the file's
>> issues.
>>
>> There is one change to take notice of: it merges two if
>> statement's conditionals together. Then it removes a
>> redundant else clause.
>
> Thanks for pointing this out, but the else statement wasn't redundant.
> Your change introduces a bug.
>
> Your patch doesn't apply.  You need to work against linux-next.
>
> regards,
> dan carpenter
>
_______________________________________________
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