diff options
author | Bram Moolenaar <Bram@vim.org> | 2018-12-22 16:49:34 +0100 |
---|---|---|
committer | Bram Moolenaar <Bram@vim.org> | 2018-12-22 16:49:34 +0100 |
commit | 9752c72f492312acd1c84e673864faed31a3bc97 (patch) | |
tree | 95f21bc103457337f8564a1bcd86216012ee0adb /src/quickfix.c | |
parent | ef3c6a5b023723a5f6eec47328cf7139c2048f8c (diff) |
patch 8.1.0622: adding quickfix items marks items as valid errorsv8.1.0622
Problem: Adding quickfix items marks items as valid errors. (Daniel Hahler)
Solution: Check when items are valid. (Yegappan Lakshmanan, closes #3683,
closes #3633)
Diffstat (limited to 'src/quickfix.c')
-rw-r--r-- | src/quickfix.c | 31 |
1 files changed, 21 insertions, 10 deletions
diff --git a/src/quickfix.c b/src/quickfix.c index 5291fb1115..71fe4939ff 100644 --- a/src/quickfix.c +++ b/src/quickfix.c @@ -6241,14 +6241,16 @@ qf_get_properties(win_T *wp, dict_T *what, dict_T *retdict) /* * Add a new quickfix entry to list at 'qf_idx' in the stack 'qi' from the - * items in the dict 'd'. + * items in the dict 'd'. If it is a valid error entry, then set 'valid_entry' + * to TRUE. */ static int qf_add_entry_from_dict( qf_info_T *qi, int qf_idx, dict_T *d, - int first_entry) + int first_entry, + int *valid_entry) { static int did_bufnr_emsg; char_u *filename, *module, *pattern, *text, *type; @@ -6313,6 +6315,9 @@ qf_add_entry_from_dict( vim_free(text); vim_free(type); + if (valid) + *valid_entry = TRUE; + return status; } @@ -6333,6 +6338,7 @@ qf_add_entries( dict_T *d; qfline_T *old_last = NULL; int retval = OK; + int valid_entry = FALSE; if (action == ' ' || qf_idx == qi->qf_listcount) { @@ -6359,22 +6365,27 @@ qf_add_entries( if (d == NULL) continue; - retval = qf_add_entry_from_dict(qi, qf_idx, d, li == list->lv_first); + retval = qf_add_entry_from_dict(qi, qf_idx, d, li == list->lv_first, + &valid_entry); if (retval == FAIL) break; } - if (qfl->qf_index == 0) + // Check if any valid error entries are added to the list. + if (valid_entry) + qfl->qf_nonevalid = FALSE; + else if (qfl->qf_index == 0) // no valid entry qfl->qf_nonevalid = TRUE; - else - qfl->qf_nonevalid = FALSE; + + // If not appending to the list, set the current error to the first entry if (action != 'a') - { qfl->qf_ptr = qfl->qf_start; - if (!qf_list_empty(qi, qf_idx)) - qfl->qf_index = 1; - } + + // Update the current error index if not appending to the list or if the + // list was empty before and it is not empty now. + if ((action != 'a' || qfl->qf_index == 0) && !qf_list_empty(qi, qf_idx)) + qfl->qf_index = 1; // Don't update the cursor in quickfix window when appending entries qf_update_buffer(qi, old_last); |