lkml.org 
[lkml]   [2012]   [Dec]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH 000/493] remove CONFIG_HOTPLUG as an option
Date
On Wed, 21 Nov 2012 20:07:23 -0500, wfp5p@viridian.itc.virginia.edu (Bill Pemberton) wrote:
> Grant Likely writes:
> >
> > You mean this series wasn't created with a script? You did this by
> > hand? If so then I must say kudos on your dedication!
> >
> > But it makes me more nervous about the series. Too easy to fat
> > finger many things when touching that many files.
> >
>
> No, I didn't do them by hand, it was a script. Originally, it was a
> couple, all basically the same, but removing each __dev*. Then I'd do
> a word diff to eyeball them to make sure the script didn't do
> something goofy.
>
> The whack-a-mole part came along because I was working against
> linux-next and whatever patch series was right for one day wouldn't be
> right for the next day because of some of the faster moving trees.
>
>
> > Please do write a script and post that for review.
> >
>
> The all-in-one version of the script:
>
> #! /usr/bin/perl
>
> use strict;
> use IO::InSitu;
>
> sub processfile
> {
> my $fn = shift;
>
> my ($in, $out) = open_rw($fn, $fn);
>
> while (<$in>) {
> s|__devexit_p\(([^)]+)\)|$1|;
> s|\s__devexit\b||;
> s|\s__devinitconst\b||;
> s|\s__devinitdata\b||;
> s|\s__devinit\b||;

Pretty straight forward, and works against the files I tried. :-)

Greg, I'd much rather see the change applied all at once in this manner.
If that isn't possible, then at the least I'll use the script against
the code that I maintain and push th result out to Linus.

g.



\
 
 \ /
  Last update: 2012-12-06 02:21    [W:0.120 / U:0.260 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site