Hi Ragavendra,

On 2015.03.14 03:04, Ragavendra Nagraj wrote:
> This patch removes the braces for the single line if statement. It fixes
> the indent positions correctly. It fixes the spaces appropriately making the 
> code give no
> warnings by the checpath.pl. Please accept.

The "Please accept." part is unnecessary. We don't want that in a commit
log forever. Also, you fix a lot of coding style issues in one commit or
IOW more than one type of change in one commit.  What you could do
instead is fix one type of coding style issue in a commit for the whole
file and do that for each coding style issue. So that would result in 3
patches instead of this one. Could you do that and resend?

[...]

-- 
Thanks,
Giedrius
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to