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

handle :GoDef from modified buffer #1132

Merged
merged 1 commit into from
Dec 17, 2016
Merged

Conversation

bhcleek
Copy link
Collaborator

@bhcleek bhcleek commented Nov 27, 2016

Handle jumping to type definitions from a modified buffer. When the
definition is in the same file, skip trying to open the file that
contains the definition - just set the cursor position. When the
definition is in a different file and the current buffer has unsaved
modifications, use :hide edit instead of :edit.

Fixes #1125

@bhcleek
Copy link
Collaborator Author

bhcleek commented Nov 27, 2016

I inadvertently made these changes on an older version of master than I should have. I'm working to fix them now, and will push a rebased commit once I have conflicts resolved.

Handle jumping to type definitions from a modified buffer. When the
definition is in the same file, skip trying to open the file that
contains the definition - just set the cursor position. When the
definition is in a different file and the current buffer has unsaved
modifications, use :hide edit instead of :edit.

Fixes fatih#1125
@bhcleek
Copy link
Collaborator Author

bhcleek commented Nov 27, 2016

I've rebased on top of current master.

@fatih fatih merged commit 0549512 into fatih:master Dec 17, 2016
@fatih
Copy link
Owner

fatih commented Dec 17, 2016

Thanks @bhcleek. Just had time to look after this. Confirmed that it was due hidden being set. This fixes it 👍

@bhcleek bhcleek deleted the fix-godef-split branch December 28, 2016 08:18
@bhcleek bhcleek mentioned this pull request Dec 28, 2016
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