[Trusty/Utopic][PATCH 1/2] x86: thinkpad_acpi.c: fixed spacing coding style issue

Andy Whitcroft apw at canonical.com
Tue Mar 10 13:43:08 UTC 2015


On Thu, Mar 05, 2015 at 04:41:12PM +0800, Adam Lee wrote:
> On Tue, Mar 03, 2015 at 03:25:36PM +0800, Adam Lee wrote:
> > On Mon, Mar 02, 2015 at 12:12:19PM +0000, Andy Whitcroft wrote:
> > > On Mon, Mar 02, 2015 at 02:40:18PM +0800, Adam Lee wrote:
> > > > From: Jan van den Berg <janvdberg at gmail.com>
> > > > 
> > > > BugLink: https://bugs.launchpad.net/bugs/1417915
> > > > 
> > > > Fixed 22 similar coding style issues: "ERROR: spaces required around that '?'"
> > > 
> > > I is not clear that is a good idea to take such a wide ranging
> > > whitespace cleanup just to get a single line of context in the second
> > > patch to match.
> > > 
> > > -apw
> > 
> > Yes, but there will be lots of patches for thinkpad_acpi in future
> > because of the sutton project, does it make sense to backport this
> > cleanup patch to avoid future context merging issues?
> > 
> > Feel free to NAK my proposal :)
> 
> Hi, Andy
> 
> How do you think? Should I submit another version?

No let it lie.  Waiting on another ack.

-apw




More information about the kernel-team mailing list