On Sat, Sep 25, 2010 at 02:31:52PM -0400, Tracey Dent wrote:
> From: Tracey Dent <Tdent48227@xxxxxxxxx>
>
> Found and corrected indent issue using checkpatch.pl
>
> Signed-off-by: Tracey Dent <tdent48227@xxxxxxxxx>
Patches that fix whitespace issues aren't really worthwhile. They
tend to cause extra work for the me as the maintainer, since it means
that patches that others send me end up failing due to whitespace
issues, which then have to be manually fixed up.
There are also changes you've made which actually make the alignment
*worse* not better, places where you changed:
} else if
to
} else if
which is to my mind, totally broken, and certainly not required by
checkpatch.
So I'm going to NACK this whole patch series, sorry.
Patches should be checkpatch.pl clean, and so as we make changes,
we'll also gradually clean up the code. Note though that I'm an emacs
user, and so I prefer code where parenthesis is properly lined up,
i.e.:
while ((count < blks) && (count <= blocks_to_boundary) &&
(le32_to_cpu(*(branch[0].p + count)) == 0)) {
and not this:
while ((count < blks) && (count <= blocks_to_boundary) &&
(le32_to_cpu(*(branch[0].p + count)) == 0)) {
I won't insist on it, though. However, I'll generally clean up such
alignment issues if I come across it.
; - Ted
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
