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

Add support for changing the current index in a quickfix/location list #3701

Closed
wants to merge 4 commits into
base: master
from

Conversation

Projects
None yet
4 participants
@yegappan
Copy link

yegappan commented Dec 17, 2018

Add the 'idx' item to the {what} argument for the setqflist() and setloclist()
functions to set the current index.

@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Dec 17, 2018

Codecov Report

Merging #3701 into master will decrease coverage by <.01%.
The diff coverage is 80.95%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3701      +/-   ##
==========================================
- Coverage   78.31%   78.31%   -0.01%     
==========================================
  Files         102      102              
  Lines      141134   141154      +20     
==========================================
+ Hits       110531   110546      +15     
- Misses      30603    30608       +5
Impacted Files Coverage Δ
src/quickfix.c 93.23% <80.95%> (-0.09%) ⬇️
src/if_xcmdsrv.c 83.99% <0%> (-0.18%) ⬇️
src/gui.c 57.93% <0%> (-0.11%) ⬇️
src/channel.c 83.2% <0%> (-0.08%) ⬇️
src/undo.c 83.11% <0%> (-0.07%) ⬇️
src/terminal.c 74.4% <0%> (-0.05%) ⬇️
src/os_unix.c 58.35% <0%> (+0.04%) ⬆️
src/ex_cmds2.c 84.59% <0%> (+0.09%) ⬆️
src/gui_gtk_x11.c 48.28% <0%> (+0.14%) ⬆️

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 4c05fa0...2496c52. Read the comment docs.

@brammool

This comment has been minimized.

Copy link
Contributor

brammool commented Dec 17, 2018

@vim-ml

This comment has been minimized.

Copy link

vim-ml commented Dec 17, 2018

@vim-ml

This comment has been minimized.

Copy link

vim-ml commented Dec 18, 2018

@yegappan yegappan force-pushed the yegappan:qfsetidx branch 3 times, most recently from 0bcd153 to 8aa6d16 Dec 22, 2018

@yegappan yegappan force-pushed the yegappan:qfsetidx branch 2 times, most recently from 269bfc2 to 2496c52 Dec 30, 2018

@yegappan yegappan force-pushed the yegappan:qfsetidx branch from 2496c52 to 255cc2f Jan 10, 2019

@brammool brammool closed this in 5b69c22 Jan 11, 2019

FONZACUS added a commit to FONZACUS/vim that referenced this pull request Jan 19, 2019

patch 8.1.0720: cannot easily change the current quickfx list index
Problem:    Cannot easily change the current quickfx list index.
Solution:   Add the "idx" argument to setqflist(). (Yegappan Lakshmanan,
            closes vim#3701)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment