Patch 8.2.1085

5 views
Skip to first unread message

Bram Moolenaar

unread,
Jun 29, 2020, 2:24:05 PM6/29/20
to vim...@googlegroups.com

Patch 8.2.1085
Problem: Coverity complains about ignoring dict_add() return value.
Solution: Add (void).
Files: src/register.c


*** ../vim-8.2.1084/src/register.c 2020-06-24 20:33:59.569106308 +0200
--- src/register.c 2020-06-29 20:22:41.087229233 +0200
***************
*** 989,1004 ****
for (n = 0; n < reg->y_size; n++)
list_append_string(list, reg->y_array[n], -1);
list->lv_lock = VAR_FIXED;
! dict_add_list(v_event, "regcontents", list);

buf[0] = (char_u)oap->regname;
buf[1] = NUL;
! dict_add_string(v_event, "regname", buf);

buf[0] = get_op_char(oap->op_type);
buf[1] = get_extra_op_char(oap->op_type);
buf[2] = NUL;
! dict_add_string(v_event, "operator", buf);

buf[0] = NUL;
buf[1] = NUL;
--- 989,1004 ----
for (n = 0; n < reg->y_size; n++)
list_append_string(list, reg->y_array[n], -1);
list->lv_lock = VAR_FIXED;
! (void)dict_add_list(v_event, "regcontents", list);

buf[0] = (char_u)oap->regname;
buf[1] = NUL;
! (void)dict_add_string(v_event, "regname", buf);

buf[0] = get_op_char(oap->op_type);
buf[1] = get_extra_op_char(oap->op_type);
buf[2] = NUL;
! (void)dict_add_string(v_event, "operator", buf);

buf[0] = NUL;
buf[1] = NUL;
***************
*** 1011,1019 ****
reglen + 1);
break;
}
! dict_add_string(v_event, "regtype", buf);

! dict_add_bool(v_event, "visual", oap->is_VIsual);

// Lock the dictionary and its keys
dict_set_items_ro(v_event);
--- 1011,1019 ----
reglen + 1);
break;
}
! (void)dict_add_string(v_event, "regtype", buf);

! (void)dict_add_bool(v_event, "visual", oap->is_VIsual);

// Lock the dictionary and its keys
dict_set_items_ro(v_event);
*** ../vim-8.2.1084/src/version.c 2020-06-29 20:20:28.047899632 +0200
--- src/version.c 2020-06-29 20:23:14.739057016 +0200
***************
*** 756,757 ****
--- 756,759 ----
{ /* Add new patch number below this line */
+ /**/
+ 1085,
/**/

--
A year spent in artificial intelligence is enough to make one
believe in God.

/// Bram Moolenaar -- Br...@Moolenaar.net -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ an exciting new programming language -- http://www.Zimbu.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
Reply all
Reply to author
Forward
0 new messages