Re: [PATCH 1/5] Combine ifs into one where possible to avoid unnecessary indentation level increase

From: Greg KH
Date: Fri Oct 03 2014 - 16:34:20 EST


On Fri, Oct 03, 2014 at 11:27:04PM +0300, Giedrius Statkevicius wrote:
> From: Giedrius StatkeviÄius <giedrius.statkevicius@xxxxxxxxx>
>
> Join together chained if's where possible to lower the indentation level.
>
> Signed-off-by: Giedrius StatkeviÄius <giedrius.statkevicius@xxxxxxxxx>

Your subject is "odd", in that it doesn't say what part of the kernel it
is for. Look at the kernel git logs, it should give you an idea of how
to format things properly.

Also, you don't have to have such a long subject:, try to keep it short,
and then be verbose in the changelog body.

Can you fix up all of these 5 and resend?

thanks,

greg k-h
--
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/