Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Unset qf_last_bufref in qf_init_ext #1728

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

Unset qf_last_bufref in qf_init_ext #1728

wants to merge 2 commits into from

Conversation

@blueyed
Copy link

@blueyed blueyed commented May 27, 2017 •
edited
Loading

Fixes #1676.

TODO:

  • test

@codecov-io
Copy link

Codecov Report

Merging #1728 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1728      +/-   ##
==========================================
+ Coverage   75.01%   75.01%   +<.01%     
==========================================
  Files          76       76              
  Lines      124946   124946              
==========================================
+ Hits        93728    93729       +1     
+ Misses      31218    31217       -1
Impacted Files Coverage Δ
src/quickfix.c 91.46% <100%> (ø) ⬆️
src/gui_beval.c 43.96% <0%> (-0.93%) ⬇️
src/message.c 68.47% <0%> (-0.06%) ⬇️
src/if_py_both.h 76.01% <0%> (-0.01%) ⬇️
src/gui_gtk_x11.c 47.33% <0%> (ø) ⬆️
src/window.c 81.28% <0%> (+0.03%) ⬆️
src/channel.c 83.67% <0%> (+0.09%) ⬆️
src/version.c 80.3% <0%> (+1.01%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b4d6c3e...9a08ad9. Read the comment docs.

@brammool brammool closed this in 6dd4a53 May 28, 2017
@blueyed blueyed deleted the fix-qf_last_bufname-on-qf_init_ext branch May 28, 2017 09:12
blueyed added a commit to blueyed/neovim that referenced this pull request May 28, 2017
@blueyed
vim-patch:8.0.0605
fed27d9 
Problem:    The buffer that quickfix caches for performance may become
            invalid. (Daniel Hahler)
Solution:   Reset qf_last_bufref in qf_init_ext(). (Daniel Hahler,
            closes vim/vim#1728, closes vim/vim#1676)

vim/vim@6dd4a53
This was referenced May 28, 2017
Merged
Closed
justinmk pushed a commit to neovim/neovim that referenced this pull request May 28, 2017
@blueyed @justinmk
vim-patch:8.0.0605 (#6821)
eb71bbb 
Problem:    The buffer that quickfix caches for performance may become
            invalid. (Daniel Hahler)
Solution:   Reset qf_last_bufref in qf_init_ext(). (Daniel Hahler,
            closes vim/vim#1728, closes vim/vim#1676)

vim/vim@6dd4a53
desvp pushed a commit to desvp/vim that referenced this pull request May 30, 2017
@brammool @desvp
patch 8.0.0605: the quickfix cached buffer may become invalid
0b90d5b 
Problem:    The buffer that quickfix caches for performance may become
            invalid. (Daniel Hahler)
Solution:   Reset qf_last_bufref in qf_init_ext(). (Daniel Hahler,
            closes vim#1728, closes vim#1676)
dpelle pushed a commit to dpelle/vim that referenced this pull request Jul 31, 2017
@brammool @dpelle
patch 8.0.0605: the quickfix cached buffer may become invalid
1cc7dc3 
Problem:    The buffer that quickfix caches for performance may become
            invalid. (Daniel Hahler)
Solution:   Reset qf_last_bufref in qf_init_ext(). (Daniel Hahler,
            closes vim#1728, closes vim#1676)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@blueyed @codecov-io