lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sun, 21 Mar 2010 16:31:26 +0100 From: Lars Lindley <lindley@...ote.org> To: Dan Carpenter <error27@...il.com>, gregkh@...e.de, greg@...ah.com, penberg@...helsinki.fi, pavel@....cz, harvey.harrison@...il.com, devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org Subject: Re: strip_whitespace.pl Yes this seems like a good tool to use. I will sure use it in the future.. /Lars On 2010-03-21 12:07, Dan Carpenter wrote: > On Sun, Mar 21, 2010 at 01:55:42PM +0300, Dan Carpenter wrote: >> I've written a new script called strip_whitespace.pl (included). >> >> One bug is that, if you split or unsplit a string literal that >> confuses it. Otherwise it seems to work. >> >> strip_whitespace.pl drivers/staging/winbond/reg.c > before >> apply patch >> strip_whitespace.pl drivers/staging/winbond/reg.c > after >> >> `diff before after`. If they are the same then resend the patch. > > On further reflection your new patch looks fine, if you combine the two. > But next time could you use my script? > > Acked-by: Dan Carpenter <error27@...il.com> > > regards, > dan carpenter > >> >> regards, >> dan carpenter >> >> #!/usr/bin/perl >> >> use strict; >> >> my $file = shift(); >> open FILE, "<$file"; >> my $txt = do { local $/; <FILE> }; >> >> # strip C99 comments >> $txt =~ s/\/\/.*//g; >> # strip newlines >> $txt =~ s/\n//g; >> # strip remaining comments >> $txt =~ s/\/\*.*?\*\///g; >> # strip tabs >> $txt =~ s/\t//g; >> # strip spaces >> $txt =~ s/ //g; >> # add newlines again >> $txt =~ s/;/;\n/g; >> >> print "$txt\n"; > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists