Re: [PATCH] checkpatch.pl: Fix warnings on code comments

From: Jeff Kirsher
Date: Mon Jan 28 2013 - 15:56:01 EST


On Mon, 2013-01-28 at 09:30 -0800, Joe Perches wrote:
> On Mon, 2013-01-28 at 17:17 +0000, Allan, Bruce W wrote:
> > David Miller Sent: Sunday, January 27, 2013 7:07 PM
> > > From: Jeff Kirsher <jeffrey.t.kirsher@xxxxxxxxx>
> > > > So will you be fine with cleanup patches which go through and
> > > > convert all the existing code comments to the desired format?
> > > Sure.
> > Not all Intel drivers...e1000e already conforms to the comment style :-)
>
> In case anyone cares, here's a perl regex
> to do network comment style conversion.
>
> $text =~ s@/\*[ \t]*\n[ \t]*\*@/*@g;
> $text =~ s@/\*([ \t]*)([^\n]+)\n[ \t]*\*/@/\*$1$2 \*/@g;
>
> (assumes the entire file is in $text)
>
> It creates a ~220KB diff for drivers/net/ethernet/intel/
> that I won't post.
>

Thanks Joe, I will get patches to take care of the Intel drivers (minus
e1000e since Bruce has already completed that work).

Attachment: signature.asc
Description: This is a digitally signed message part