lkml.org 
[lkml]   [2013]   [Jul]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] firmware/dmi_scan: Drop OOM messages
From
Date
On Wed, 2013-07-10 at 18:17 +0200, Jean Delvare wrote:
> Le Wednesday 10 July 2013 à 07:51 -0700, Joe Perches a écrit :
> > even more trivial...
[]
> > It'd also be nice to use the same test style
> > in all 6 uses.
> >
> > if (!foo)
>
> Both styles are used throughout the file, and checkpatch doesn't
> complain.

And it might be wrong for checkpatch to complain as
there are tens of thousands of those in the kernel.

I think using both styles in the same file is poor
form though.

> Unifying the coding style is beyond the scope of my patchset,
> sorry. Feel free to send a patch on top of mine if you really care.

No worries. If I remember, I'll send a patch after
yours hit -next.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2013-07-11 00:02    [W:0.039 / U:0.764 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site