diff options
| author | Bram Moolenaar <Bram@vim.org> | 2020-07-01 13:04:05 +0200 |
|---|---|---|
| committer | Bram Moolenaar <Bram@vim.org> | 2020-07-01 13:04:05 +0200 |
| commit | 90049492215aa458b90215b8e0fc50f04d5ad270 (patch) | |
| tree | 4a446594aa83271f92f0089373caab8a4b92e943 /src/quickfix.c | |
| parent | be7a50c22f63478a6e64fe6b932a847830191b95 (diff) | |
| download | vim-git-90049492215aa458b90215b8e0fc50f04d5ad270.tar.gz | |
patch 8.2.1102: Coverity gets confused by an unnecessary NULL checkv8.2.1102
Problem: Coverity gets confused by an unnecessary NULL check.
Solution: Remove the check for NULL.
Diffstat (limited to 'src/quickfix.c')
| -rw-r--r-- | src/quickfix.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/src/quickfix.c b/src/quickfix.c index 091246b76..a7d211822 100644 --- a/src/quickfix.c +++ b/src/quickfix.c @@ -7374,6 +7374,7 @@ qf_free_stack(win_T *wp, qf_info_T *qi) * Populate the quickfix list with the items supplied in the list * of dictionaries. "title" will be copied to w:quickfix_title. * "action" is 'a' for add, 'r' for replace. Otherwise create a new list. + * When "what" is not NULL then only set some properties. */ int set_errorlist( @@ -7401,7 +7402,7 @@ set_errorlist( } // A dict argument cannot be specified with a non-empty list argument - if (list != NULL && list->lv_len != 0 && what != NULL) + if (list->lv_len != 0 && what != NULL) { semsg(_(e_invarg2), _("cannot have both a list and a \"what\" argument")); @@ -8106,7 +8107,7 @@ set_qf_ll_list( else { list_T *l = list_arg->vval.v_list; - dict_T *d = NULL; + dict_T *what = NULL; int valid_dict = TRUE; if (action_arg->v_type == VAR_STRING) @@ -8128,8 +8129,8 @@ set_qf_ll_list( if (action_arg->v_type != VAR_UNKNOWN && what_arg->v_type != VAR_UNKNOWN) { - if (what_arg->v_type == VAR_DICT) - d = what_arg->vval.v_dict; + if (what_arg->v_type == VAR_DICT && what_arg->vval.v_dict != NULL) + what = what_arg->vval.v_dict; else { emsg(_(e_dictreq)); @@ -8138,9 +8139,10 @@ set_qf_ll_list( } ++recursive; - if (l != NULL && action && valid_dict && set_errorlist(wp, l, action, + if (l != NULL && action && valid_dict + && set_errorlist(wp, l, action, (char_u *)(wp == NULL ? ":setqflist()" : ":setloclist()"), - d) == OK) + what) == OK) rettv->vval.v_number = 0; --recursive; } |