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

wrong suggestions were given, remove algorithm that seeks for first pare... #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

noypi
Copy link

@noypi noypi commented Oct 21, 2014

GOnpp was not giving me the suggestions properly in my go source code. It was always sending the offset 469 to gocode. The true offset is within a closure, which was a parameter of a function from the offset 469.

Since gocode already offers the best suggestions base on the provided offset, can we just remove the algorithm that seeks for the parenthesis?

  • Thanks.

unknown added 2 commits October 21, 2014 20:34
…arenthesis. gocode already offers best suggestions base on offset.
…ide in GOROOT/bin and some configurations will have GOBIN in some directory. GOBIN is where commands are installed and may not be where go.exe is. The official go documentation may say that the go compiler resides in GOBIN, i do not know -- it's much better to assume that the go compiler resides in GOROOT/bin.
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.

1 participant