Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use wipe instead of delete on hidden? #26

Open
sodapopcan opened this issue Mar 7, 2014 · 3 comments
Open

Use wipe instead of delete on hidden? #26

sodapopcan opened this issue Mar 7, 2014 · 3 comments

Comments

@sodapopcan
Copy link

I always get this error when I use c-o or ^ after quitting extradite which is incredibly frustrating:

Error detected while processing CursorMoved Auto commands for
"<buffer=35>":                                      
E121: Undefined variable: b:extradata_list

I fix it by changing bufhidden=delete to bufhidden=wipe on line 139. This, of course, wipes out the alternate file for ^ no longer works, but it's much better than getting the error and being taken to the commit log (minus the preview window). C-o still works fine.

Any other ideas on getting around this?

@xaizek
Copy link
Collaborator

xaizek commented Mar 9, 2014

Any other ideas on getting around this?

I though using :keepalt modifier can help, but it doesn't. So there seems to be no better alternative to your solution.

Thanks for the suggestion!

@sodapopcan
Copy link
Author

NERD tree does some sorcery to get around this but haven't put in the time to figure this out. I too tried keepalt to no avail :\ Other than doing something hacky like saving the current alt file to a variant, the jumping to it then back to previous file after quit, I'm not sure what else could be done here other than wiping the buffer completely. I certainly prefer this to getting the error as it happens frequently. Could fix the error too, of course, but I'd rather not be taken back to the Extradite buffer.

xaizek added a commit that referenced this issue Mar 12, 2014
This way alternative file history is lost, but jumping around using
Ctrl-I and Ctrl-O works as expected.  See #26 for more details.
@xaizek
Copy link
Collaborator

xaizek commented Mar 12, 2014

I've pushed the change of bufhidden to wipe as it's the best we have for now. I didn't find anything interesting in closeTreeWindow() function of NERDTree.

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

No branches or pull requests

2 participants