On Lindent shortcomings and massive style fixing

From	Andrey Utkin <andrey.utkin@corp.bluecherry.net>
Newsgroups	linux.kernel
Subject	On Lindent shortcomings and massive style fixing
Date	2015-12-28 15:40 +0100
Message-ID	<qKH7Q-3lo-11@gated-at.bofh.it> (permalink)
Organization	linux.* mail to news gateway

Show all headers | View raw

After some iterations of checkpatch.pl, on a new developed driver
(tw5864), now I have the following:

 $ grep 'WARNING\|ERROR' /src/checkpatch.tw5864 | sort | uniq -c
     31 ERROR: do not use C99 // comments
    147 WARNING: Block comments use a trailing */ on a separate line
    144 WARNING: Block comments use * on subsequent lines
    435 WARNING: line over 80 characters

At this point, Lindent was already used, and checkpatch.pl warnings
introduced by Lindent itself were fixed. Usage of "indent
--linux-style" (which behaves differently BTW) doesn't help anymore,
too.

Could anybody please advise how to sort out these issues
automatically, because they look like perfectly solvable in automated
fashion. Of course manual work would result in more niceness, but I am
not eager to go through hundreds of place of code just to fix "over 80
characters" issues now.

First one ("do not use C99 // comments") looks easy with regexps, but
the other are not.

Is there any known improvements or successors for Lindent? Or could we
get indent/Lindent improved if we collect some money for this task?

If anybody wants to look at actual code, here it is:
https://github.com/bluecherrydvr/linux.git , branch tw5864_stable,
drivers/staging/media/tw5864

Current output of "checkpatch.pl -f" for all source files in the
driver is here:
https://gist.github.com/andrey-utkin/12295148475e34ef948b

Thanks in advance.
--
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/

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

On Lindent shortcomings and massive style fixing Andrey Utkin <andrey.utkin@corp.bluecherry.net> - 2015-12-28 15:40 +0100
  Re: On Lindent shortcomings and massive style fixing Greg KH <greg@kroah.com> - 2015-12-28 16:40 +0100
    Re: On Lindent shortcomings and massive style fixing Mauro Carvalho Chehab <mchehab@osg.samsung.com> - 2015-12-29 08:40 +0100
      Re: On Lindent shortcomings and massive style fixing Andrey Utkin <andrey.utkin@corp.bluecherry.net> - 2015-12-29 10:20 +0100
        Re: On Lindent shortcomings and massive style fixing Julia Lawall <julia.lawall@lip6.fr> - 2015-12-29 11:20 +0100

csiph-web