On Thu, Dec 06, 2012 at 12:27:42AM +0000, Grant Likely wrote: > On Wed, 21 Nov 2012 20:07:23 -0500, wf...@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.
Given that there are a lot of patches already in linux-next from Bill due to this work, I'm not going to do this for all files right now, sorry. But, if you want to use this for the files you maintain and push that out for 3.8-rc1, that would be great. I'll be walking the tree after 3.8-rc1 is out to catch the stragglers with a script like this. thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/