lkml.org 
[lkml]   [2011]   [Feb]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: rename_rev.pl script for reviewing renames
From
Date
On Sun, 2011-02-06 at 15:25 +0300, Dan Carpenter wrote:
> Or if you have a camel case script that changes "ThisVariable" to
> "this_variable". Then the command would be:
> git show c659c38 | ./rename_rev.pl -ea '$_ = lc' -ea 's/_//g'
> Which changes everything to lower case and strips out all the
> underscores. You might want to combine it with some other flags:
> git show c659c38 | ./rename_rev.pl -nc \
> -e 's/TLanPrivateInfo/struct tlan_priv/' \
> -e 's/TLanList/struct tlan_list/' \
> -ea '$_ = lc' -ea 's/_//g'

Hi Dan.

I think you'll need to add '\b$1\b' to your tests
otherwise your first example will check things like
"TLanPrivateInfoType" as well.

> What I would like is if there was some way to ignore changes which just
> introduced new lines, but didn't affect runtime behavior. I'm not sure
> how to do that.

Any object change will affect runtime behavior.




\
 
 \ /
  Last update: 2011-02-07 04:41    [W:0.464 / U:0.336 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site