Call to list_append_dict cannot fail.

Clean up the use of list_append_dict and remove error checks.
This commit is contained in:
oni-link 2014-04-24 15:41:45 +02:00 committed by Thiago de Arruda
parent 9da32a1f78
commit 8d5a546d3f
4 changed files with 5 additions and 14 deletions

View File

@ -5535,20 +5535,16 @@ int list_append_tv(list_T *l, typval_T *tv)
/*
* Add a dictionary to a list. Used by getqflist().
* Return FAIL when out of memory.
*/
int list_append_dict(list_T *list, dict_T *dict)
void list_append_dict(list_T *list, dict_T *dict)
{
listitem_T *li = listitem_alloc();
if (li == NULL)
return FAIL;
li->li_tv.v_type = VAR_DICT;
li->li_tv.v_lock = 0;
li->li_tv.vval.v_dict = dict;
list_append(list, li);
++dict->dv_refcount;
return OK;
}
/*

View File

@ -64,7 +64,7 @@ listitem_T *list_find(list_T *l, long n);
char_u *list_find_str(list_T *l, long idx);
void list_append(list_T *l, listitem_T *item);
int list_append_tv(list_T *l, typval_T *tv);
int list_append_dict(list_T *list, dict_T *dict);
void list_append_dict(list_T *list, dict_T *dict);
int list_append_string(list_T *l, char_u *str, int len);
int list_insert_tv(list_T *l, typval_T *tv, listitem_T *item);
void list_remove(list_T *l, listitem_T *item, listitem_T *item2);

View File

@ -3287,8 +3287,7 @@ int get_errorlist(win_T *wp, list_T *list)
if ((dict = dict_alloc()) == NULL)
return FAIL;
if (list_append_dict(list, dict) == FAIL)
return FAIL;
list_append_dict(list, dict);
buf[0] = qfp->qf_type;
buf[1] = NUL;

View File

@ -789,10 +789,7 @@ do_tag (
if ((dict = dict_alloc()) == NULL)
continue;
if (list_append_dict(list, dict) == FAIL) {
vim_free(dict);
continue;
}
list_append_dict(list, dict);
dict_add_nr_str(dict, "text", 0L, tag_name);
dict_add_nr_str(dict, "filename", 0L, fname);
@ -2878,8 +2875,7 @@ int get_tags(list_T *list, char_u *pat)
if ((dict = dict_alloc()) == NULL)
ret = FAIL;
if (list_append_dict(list, dict) == FAIL)
ret = FAIL;
list_append_dict(list, dict);
full_fname = tag_full_fname(&tp);
if (add_tag_field(dict, "name", tp.tagname, tp.tagname_end) == FAIL