lkml.org 
[lkml]   [2011]   [Mar]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH]Clean up CodingStyle for scripts/kconfig/gconf.c

>>> Hi us,
>>> Clean up CodingStyle for scripts/kconfig/gconf.c.
>>>
>>
>> What makes you think this is a bad style? Does Documentation/CodingStyle say
>> so? Does checkpatch.pl make a complain? Do you really think this improves
>> readability? I don't think so, and actually it makes the code worse.
> But we all write '#include <***>'. I have never seen '# include <***>'.

Then let's see:

$ find -name '*.c' -exec grep -H '^# .*include' {} \;
./drivers/char/mem.c:# include <linux/efi.h>
./drivers/char/random.c:# include <linux/irq.h>
./drivers/scsi/nsp32.c:# include "nsp32_debug.c"
./drivers/video/epson1355fb.c:# include <mach/hardware.h>

You can look into those cases, and then you should be able to figure out
why.

>>
>> Run checkpatch.pl on this file, and you'll see lots of errors and warnings,
>> but pure codingstyle cleanup is rarely worth anything, so better don't do
>> that.
> I am sorry, I have not got your meaning clearly.

Codingstyle cleanups aren't worth your time, and it would be better for you
and for the kernel project as a whole if you can find something more valuable
to work on.

>>
>>> Thanks.
>>> Best Regards.
>>> Harry Wei.
>>>
>>> Signed-off-by: Harry Wei <harryxiyou@gmail.com>
>>> ---
>>> scripts/kconfig/gconf.c | 2 +-
>>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>>
>>> diff --git a/scripts/kconfig/gconf.c b/scripts/kconfig/gconf.c
>>> index 4558961..56da945 100644
>>> --- a/scripts/kconfig/gconf.c
>>> +++ b/scripts/kconfig/gconf.c
>>> @@ -7,7 +7,7 @@
>>> */
>>>
>>> #ifdef HAVE_CONFIG_H
>>> -# include <config.h>
>>> +#include <config.h>
>>> #endif
>>>
>>> #include "lkc.h"


\
 
 \ /
  Last update: 2011-03-23 08:33    [W:0.042 / U:2.088 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site