mirror of
https://github.com/neovim/neovim.git
synced 2025-02-25 18:55:25 -06:00
vim-patch:8.0.1384: not enough quickfix help; confusing winid
Problem: Not enough quickfix help; confusing winid.
Solution: Add more examples in the help. When the quickfix window is not
present, return zero for getqflist() with 'winid'. Add more tests
for jumping to quickfix list entries. (Yegappan Lakshmanan, closes
vim/vim#2427)
74240d3feb
This commit is contained in:
Jan Edmund Lazo
parent
8e8c7d7544
commit
773c488140
@ -28,7 +28,7 @@ func s:setup_commands(cchar)
|
||||
command! -count -nargs=* -bang Xprev <mods><count>cprev<bang> <args>
|
||||
command! -nargs=* -bang Xfirst <mods>cfirst<bang> <args>
|
||||
command! -nargs=* -bang Xlast <mods>clast<bang> <args>
|
||||
command! -nargs=* -bang Xnfile <mods>cnfile<bang> <args>
|
||||
command! -nargs=* -bang -range Xnfile <mods><count>cnfile<bang> <args>
|
||||
command! -nargs=* -bang Xpfile <mods>cpfile<bang> <args>
|
||||
command! -nargs=* Xexpr <mods>cexpr <args>
|
||||
command! -range -nargs=* Xvimgrep <mods><count>vimgrep <args>
|
||||
@ -36,6 +36,7 @@ func s:setup_commands(cchar)
|
||||
command! -nargs=* Xgrep <mods> grep <args>
|
||||
command! -nargs=* Xgrepadd <mods> grepadd <args>
|
||||
command! -nargs=* Xhelpgrep helpgrep <args>
|
||||
command! -nargs=0 -count Xcc <count>cc
|
||||
let g:Xgetlist = function('getqflist')
|
||||
let g:Xsetlist = function('setqflist')
|
||||
call setqflist([], 'f')
|
||||
@ -60,7 +61,7 @@ func s:setup_commands(cchar)
|
||||
command! -count -nargs=* -bang Xprev <mods><count>lprev<bang> <args>
|
||||
command! -nargs=* -bang Xfirst <mods>lfirst<bang> <args>
|
||||
command! -nargs=* -bang Xlast <mods>llast<bang> <args>
|
||||
command! -nargs=* -bang Xnfile <mods>lnfile<bang> <args>
|
||||
command! -nargs=* -bang -range Xnfile <mods><count>lnfile<bang> <args>
|
||||
command! -nargs=* -bang Xpfile <mods>lpfile<bang> <args>
|
||||
command! -nargs=* Xexpr <mods>lexpr <args>
|
||||
command! -range -nargs=* Xvimgrep <mods><count>lvimgrep <args>
|
||||
@ -68,6 +69,7 @@ func s:setup_commands(cchar)
|
||||
command! -nargs=* Xgrep <mods> lgrep <args>
|
||||
command! -nargs=* Xgrepadd <mods> lgrepadd <args>
|
||||
command! -nargs=* Xhelpgrep lhelpgrep <args>
|
||||
command! -nargs=0 -count Xcc <count>ll
|
||||
let g:Xgetlist = function('getloclist', [0])
|
||||
let g:Xsetlist = function('setloclist', [0])
|
||||
call setloclist(0, [], 'f')
|
||||
@ -395,12 +397,18 @@ endfunc
|
||||
func Xtest_browse(cchar)
|
||||
call s:setup_commands(a:cchar)
|
||||
|
||||
call g:Xsetlist([], 'f')
|
||||
" Jumping to first or next location list entry without any error should
|
||||
" result in failure
|
||||
if a:cchar == 'l'
|
||||
call assert_fails('lfirst', 'E776:')
|
||||
call assert_fails('lnext', 'E776:')
|
||||
if a:cchar == 'c'
|
||||
let err = 'E42:'
|
||||
else
|
||||
let err = 'E776:'
|
||||
endif
|
||||
call assert_fails('Xnext', err)
|
||||
call assert_fails('Xprev', err)
|
||||
call assert_fails('Xnfile', err)
|
||||
call assert_fails('Xpfile', err)
|
||||
|
||||
call s:create_test_file('Xqftestfile1')
|
||||
call s:create_test_file('Xqftestfile2')
|
||||
@ -421,6 +429,12 @@ func Xtest_browse(cchar)
|
||||
Xpfile
|
||||
call assert_equal('Xqftestfile1', bufname('%'))
|
||||
call assert_equal(6, line('.'))
|
||||
5Xcc
|
||||
call assert_equal(5, g:Xgetlist({'idx':0}).idx)
|
||||
2Xcc
|
||||
call assert_equal(2, g:Xgetlist({'idx':0}).idx)
|
||||
10Xcc
|
||||
call assert_equal(6, g:Xgetlist({'idx':0}).idx)
|
||||
Xlast
|
||||
Xprev
|
||||
call assert_equal('Xqftestfile2', bufname('%'))
|
||||
@ -438,6 +452,23 @@ func Xtest_browse(cchar)
|
||||
call assert_equal('Xqftestfile1', bufname('%'))
|
||||
call assert_equal(5, line('.'))
|
||||
|
||||
" Jumping to an error from the error window using cc command
|
||||
Xgetexpr ['Xqftestfile1:5:Line5',
|
||||
\ 'Xqftestfile1:6:Line6',
|
||||
\ 'Xqftestfile2:10:Line10',
|
||||
\ 'Xqftestfile2:11:Line11']
|
||||
Xopen
|
||||
10Xcc
|
||||
call assert_equal(11, line('.'))
|
||||
call assert_equal('Xqftestfile2', bufname('%'))
|
||||
|
||||
" Jumping to an error from the error window (when only the error window is
|
||||
" present)
|
||||
Xopen | only
|
||||
Xlast 1
|
||||
call assert_equal(5, line('.'))
|
||||
call assert_equal('Xqftestfile1', bufname('%'))
|
||||
|
||||
Xexpr ""
|
||||
call assert_fails('Xnext', 'E42:')
|
||||
|
||||
@ -1512,13 +1543,18 @@ func Test_switchbuf()
|
||||
set switchbuf=usetab
|
||||
tabedit Xqftestfile1
|
||||
tabedit Xqftestfile2
|
||||
tabedit Xqftestfile3
|
||||
tabfirst
|
||||
cfirst | cnext
|
||||
call assert_equal(2, tabpagenr())
|
||||
2cnext
|
||||
call assert_equal(3, tabpagenr())
|
||||
2cnext
|
||||
call assert_equal(3, tabpagenr())
|
||||
6cnext
|
||||
call assert_equal(4, tabpagenr())
|
||||
2cpfile
|
||||
call assert_equal(2, tabpagenr())
|
||||
2cnfile
|
||||
call assert_equal(4, tabpagenr())
|
||||
tabfirst | tabonly | enew
|
||||
|
||||
set switchbuf=split
|
||||
@ -2338,7 +2374,7 @@ func XfreeTests(cchar)
|
||||
Xclose
|
||||
endfunc
|
||||
|
||||
" Tests for the quickifx free functionality
|
||||
" Tests for the quickfix free functionality
|
||||
func Test_qf_free()
|
||||
call XfreeTests('c')
|
||||
call XfreeTests('l')
|
||||
@ -3165,3 +3201,100 @@ func Test_lvimgrep_crash()
|
||||
augroup END
|
||||
enew | only
|
||||
endfunc
|
||||
|
||||
func Xqfjump_tests(cchar)
|
||||
call s:setup_commands(a:cchar)
|
||||
|
||||
call writefile(["Line1\tFoo", "Line2"], 'F1')
|
||||
call writefile(["Line1\tBar", "Line2"], 'F2')
|
||||
call writefile(["Line1\tBaz", "Line2"], 'F3')
|
||||
|
||||
call g:Xsetlist([], 'f')
|
||||
|
||||
" Tests for
|
||||
" Jumping to a line using a pattern
|
||||
" Jumping to a column greater than the last column in a line
|
||||
" Jumping to a line greater than the last line in the file
|
||||
let l = []
|
||||
for i in range(1, 7)
|
||||
call add(l, {})
|
||||
endfor
|
||||
let l[0].filename='F1'
|
||||
let l[0].pattern='Line1'
|
||||
let l[1].filename='F2'
|
||||
let l[1].pattern='Line1'
|
||||
let l[2].filename='F3'
|
||||
let l[2].pattern='Line1'
|
||||
let l[3].filename='F3'
|
||||
let l[3].lnum=1
|
||||
let l[3].col=9
|
||||
let l[3].vcol=1
|
||||
let l[4].filename='F3'
|
||||
let l[4].lnum=99
|
||||
let l[5].filename='F3'
|
||||
let l[5].lnum=1
|
||||
let l[5].col=99
|
||||
let l[5].vcol=1
|
||||
let l[6].filename='F3'
|
||||
let l[6].pattern='abcxyz'
|
||||
|
||||
call g:Xsetlist([], ' ', {'items' : l})
|
||||
Xopen | only
|
||||
2Xnext
|
||||
call assert_equal(3, g:Xgetlist({'idx' : 0}).idx)
|
||||
call assert_equal('F3', bufname('%'))
|
||||
Xnext
|
||||
call assert_equal(7, col('.'))
|
||||
Xnext
|
||||
call assert_equal(2, line('.'))
|
||||
Xnext
|
||||
call assert_equal(9, col('.'))
|
||||
2
|
||||
Xnext
|
||||
call assert_equal(2, line('.'))
|
||||
|
||||
if a:cchar == 'l'
|
||||
" When jumping to a location list entry in the location list window and
|
||||
" no usable windows are available, then a new window should be opened.
|
||||
enew! | new | only
|
||||
call g:Xsetlist([], 'f')
|
||||
setlocal buftype=nofile
|
||||
new
|
||||
call g:Xsetlist([], ' ', {'lines' : ['F1:1:1:Line1', 'F1:2:2:Line2', 'F2:1:1:Line1', 'F2:2:2:Line2', 'F3:1:1:Line1', 'F3:2:2:Line2']})
|
||||
Xopen
|
||||
let winid = win_getid()
|
||||
wincmd p
|
||||
close
|
||||
call win_gotoid(winid)
|
||||
Xnext
|
||||
call assert_equal(3, winnr('$'))
|
||||
call assert_equal(1, winnr())
|
||||
call assert_equal(2, line('.'))
|
||||
|
||||
" When jumping to an entry in the location list window and the window
|
||||
" associated with the location list is not present and a window containing
|
||||
" the file is already present, then that window should be used.
|
||||
close
|
||||
belowright new
|
||||
call g:Xsetlist([], 'f')
|
||||
edit F3
|
||||
call win_gotoid(winid)
|
||||
Xlast
|
||||
call assert_equal(3, winnr())
|
||||
call assert_equal(6, g:Xgetlist({'size' : 1}).size)
|
||||
call assert_equal(winid, g:Xgetlist({'winid' : 1}).winid)
|
||||
endif
|
||||
|
||||
" Cleanup
|
||||
enew!
|
||||
new | only
|
||||
|
||||
call delete('F1')
|
||||
call delete('F2')
|
||||
call delete('F3')
|
||||
endfunc
|
||||
|
||||
func Test_qfjump()
|
||||
call Xqfjump_tests('c')
|
||||
call Xqfjump_tests('l')
|
||||
endfunc
|
||||
|
||||
Loading…
Reference in New Issue
Block a user