Re: [PATCH 1/9] staging: sm750fb: fix line length coding style issue in ddk750_chip.c
From: Moshe Green <hidden>
Date: 2016-09-03 19:58:09
Also in:
linux-fbdev
From: Moshe Green <hidden>
Date: 2016-09-03 19:58:09
Also in:
linux-fbdev
On Fri, Sep 02, 2016 at 01:35:08PM +0200, Greg KH wrote:
A: Because it messes up the order in which people normally read text. Q: Why is top-posting such a bad thing? A: Top-posting. Q: What is the most annoying thing in e-mail? A: No. Q: Should I include quotations after my reply? http://daringfireball.net/2007/07/on_top On Fri, Sep 02, 2016 at 11:28:51AM +0300, moshe green wrote:quoted
There were (mostly) two kinds of warnings that I've handled, "80 character line length" and block comments structure. Some of the changes involve handling two separate warnings in a single comment. Where this occurs, should I break down the change into two steps - fixing a warning at a time? Or should I fix both warnings in the same commit - and place the commit in the most appropriate patch?Hm, use your best judgement here, what would you want to be on the receiving end of if you had to review such a patch series? thanks, greg k-h
Will do. thanks, Moshe Green