diff options
author | Bram Moolenaar <Bram@vim.org> | 2018-08-09 21:19:20 +0200 |
---|---|---|
committer | Bram Moolenaar <Bram@vim.org> | 2018-08-09 21:19:20 +0200 |
commit | 3f347e4716c44cf6458be407e712e3d708d82580 (patch) | |
tree | 7db691b28433b1f4211f4b0e942c196ffc027e30 /src | |
parent | af559d2c9f44bc88a7d94f9236b3c024563a8e73 (diff) |
patch 8.1.0259: no test for fixed quickfix issuev8.1.0259
Problem: No test for fixed quickfix issue.
Solution: Add a test. Clean up the code a bit. (Yegappan Lakshmanan)
Diffstat (limited to 'src')
-rw-r--r-- | src/quickfix.c | 12 | ||||
-rw-r--r-- | src/testdir/test_quickfix.vim | 24 | ||||
-rw-r--r-- | src/version.c | 2 |
3 files changed, 29 insertions, 9 deletions
diff --git a/src/quickfix.c b/src/quickfix.c index d4aa5090da..14fcfcff43 100644 --- a/src/quickfix.c +++ b/src/quickfix.c @@ -1506,7 +1506,6 @@ qf_list_empty(qf_info_T *qi, int qf_idx) return qi->qf_lists[qf_idx].qf_count <= 0; } - /* * Allocate the fields used for parsing lines and populating a quickfix list. */ @@ -3717,7 +3716,7 @@ qf_view_result(int split) if (IS_LL_WINDOW(curwin)) qi = GET_LOC_LIST(curwin); - if (qi == NULL || qi->qf_lists[qi->qf_curlist].qf_count == 0) + if (qf_list_empty(qi, qi->qf_curlist)) { EMSG(_(e_quickfix)); return; @@ -4349,7 +4348,8 @@ qf_jump_first(qf_info_T *qi, int_u save_qfid, int forceit) if (qi->qf_lists[qi->qf_curlist].qf_id != save_qfid) qi->qf_curlist = qf_id2nr(qi, save_qfid); - if (qi->qf_lists[qi->qf_curlist].qf_count > 0) + // Autocommands might have cleared the list, check for it + if (!qf_list_empty(qi, qi->qf_curlist)) qf_jump(qi, 0, 0, forceit); } @@ -4873,10 +4873,8 @@ ex_cfile(exarg_T *eap) // free the list. if (res > 0 && (eap->cmdidx == CMD_cfile || eap->cmdidx == CMD_lfile) && qflist_valid(wp, save_qfid)) - { // display the first error qf_jump_first(qi, save_qfid, eap->forceit); - } } /* @@ -6581,10 +6579,8 @@ ex_cbuffer(exarg_T *eap) if (res > 0 && (eap->cmdidx == CMD_cbuffer || eap->cmdidx == CMD_lbuffer) && qflist_valid(wp, save_qfid)) - { // display the first error qf_jump_first(qi, save_qfid, eap->forceit); - } } } } @@ -6661,10 +6657,8 @@ ex_cexpr(exarg_T *eap) if (res > 0 && (eap->cmdidx == CMD_cexpr || eap->cmdidx == CMD_lexpr) && qflist_valid(wp, save_qfid)) - { // display the first error qf_jump_first(qi, save_qfid, eap->forceit); - } } else EMSG(_("E777: String or List expected")); diff --git a/src/testdir/test_quickfix.vim b/src/testdir/test_quickfix.vim index d41815f3ce..ce19e74d3f 100644 --- a/src/testdir/test_quickfix.vim +++ b/src/testdir/test_quickfix.vim @@ -3478,6 +3478,30 @@ func Xautocmd_changelist(cchar) call assert_equal(5, line('.')) autocmd! QuickFixCmdPost + " Test for autocommands clearing the quickfix list before jumping to the + " first error. This should not result in an error + autocmd QuickFixCmdPost * call g:Xsetlist([], 'r') + let v:errmsg = '' + " Test for cfile/lfile + Xfile Xerr + call assert_true(v:errmsg !~# 'E42:') + " Test for cbuffer/lbuffer + edit Xerr + Xbuffer + call assert_true(v:errmsg !~# 'E42:') + " Test for cexpr/lexpr + Xexpr 'Xtestfile2:4:Line4' + call assert_true(v:errmsg !~# 'E42:') + " Test for grep/lgrep + " The grepprg may not be set on non-Unix systems + if has('unix') + silent Xgrep Line5 Xtestfile2 + call assert_true(v:errmsg !~# 'E42:') + endif + " Test for vimgrep/lvimgrep + call assert_fails('silent Xvimgrep Line5 Xtestfile2', 'E480:') + autocmd! QuickFixCmdPost + call delete('Xerr') call delete('Xtestfile1') call delete('Xtestfile2') diff --git a/src/version.c b/src/version.c index 4f31d66e87..dadb978b31 100644 --- a/src/version.c +++ b/src/version.c @@ -795,6 +795,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 259, +/**/ 258, /**/ 257, |