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

Quickfix window is no longer closed on Vim 7.4 #1423

Closed
cespare opened this issue Sep 1, 2017 · 4 comments
Closed

Quickfix window is no longer closed on Vim 7.4 #1423

cespare opened this issue Sep 1, 2017 · 4 comments
Labels

Comments

@cespare
Copy link
Contributor

cespare commented Sep 1, 2017

After 9a433e5 and the follow-up fix 52c5b1f, the quickfix window is no longer closed in Vim 7.4.

To reproduce:

  1. Open some go file.
  2. Make it invalid by introducing a syntax error.
  3. Save, so that a gofmt error appears in the quickfix window.
  4. Fix the syntax error.
  5. Save again, and observe that the quickfix window doesn't go away.

The particular vim I was using gave this for vim --version:

VIM - Vi IMproved 7.4 (2013 Aug 10, compiled Nov 24 2016 16:44:48)
Included patches: 1-1689
Extra patches: 8.0.0056
<lots of other junk>

The particular behaviors I observed were:

  1. Before 9a433e5: the quickfix window is closed when I expect.
  2. After 9a433e5: there's some crash regarding getqflist when the quickfix window is supposed to be closed.
  3. After 52c5b1f: there's no crash, but the quickfix window isn't closed.

I upgraded to vim 8.0.1033 and then this fixed the problem (now the quickfix window is closed as expected).

So basically, I think that 9a433e5 needs to be adjusted for vim 7 compatibility.

@michaelyou
Copy link

I have also encountered this problem

@psaia
Copy link

psaia commented Sep 24, 2017

I can confirm as well. It appeared after an update.

@bhcleek
Copy link
Collaborator

bhcleek commented Sep 24, 2017

@cespare @michaelyou @petesaia As a workaround you can also :lclose or :cclose (depending on your settings for the list to use for :GoFmt.

However, I think that #1474 will fix your issue. Can you confirm?

@jthurman42
Copy link

#1474 doesn't resolve this if you do not have patch 2200, but does resolve it with an additional line (commented in that PR)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants