Skip to content

Conversation

nrocy
Copy link
Collaborator

@nrocy nrocy commented Oct 10, 2010

Hey!

Does what it says on the tin, I've merged upstream already so should be an easy merge for you :)

Also there's a technique in there for backing up @registers to avoid clobbering them, according to #vim on FreeNode, you can't yank into local variables. :(

  • Paul

@ecomba
Copy link
Owner

ecomba commented Oct 10, 2010

Ooops!

Error detected while processing function RenameLocalVariable:
line 29:
E486: Pattern not found: <\zssomething>\ze[^(]

@nrocy
Copy link
Collaborator Author

nrocy commented Oct 10, 2010

Oops indeed!

Was it a variable in a method, or in global scope? Got a snippet/steps to reproduce please?

@ecomba
Copy link
Owner

ecomba commented Oct 10, 2010

I had this method here:

26 def arg
27 something = 3+4
28 bla + somethig
29 return something
30 end

Selected in visual mode the first occurrence of "something" and tried to run the script rrlv

@nrocy
Copy link
Collaborator Author

nrocy commented Oct 10, 2010

Hey Enrique,

Have added a dependency check into the plugin for now, it's actually fairly hard to match the corresponding end token. As such, figure better safe than sorry :) Gives an error message stating requirement, and where to download plugin from.

  • Paul

@ecomba
Copy link
Owner

ecomba commented Oct 11, 2010

Well done!

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants