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

go#package#Paths more efficient at getting GOROOT #702

Merged
merged 1 commit into from
Jan 28, 2016
Merged

go#package#Paths more efficient at getting GOROOT #702

merged 1 commit into from
Jan 28, 2016

Conversation

nhooyr
Copy link
Contributor

@nhooyr nhooyr commented Jan 27, 2016

go#package#Paths for some reason would call go env GOROOT every single
time it needed the GOROOT. In #697 the problem was that it was calling
go to get the GOROOT every single time the statusline needed an update
which resulted in the flashing cursor. Now it only calls it once and
stores it in a script local variable.

go#package#Paths for some reason would call go env GOROOT every single
time it needed the GOROOT. In #697 the problem was that it was calling
go to get the GOROOT every single time the statusline needed an update
which resulted in the flashing cursor. Now it only calls it once and
stores it in a script local variable.
fatih added a commit that referenced this pull request Jan 28, 2016
go#package#Paths more efficient at getting GOROOT
@fatih fatih merged commit a1cfa11 into fatih:master Jan 28, 2016
@fatih
Copy link
Owner

fatih commented Jan 28, 2016

Thanks @nhooyr. Great fix 👍

@nhooyr nhooyr deleted the efficient-goroot branch January 28, 2016 15:29
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