neovim

commit 30604320072335122aea0f37890f136b2ba0e445
parent e8cc489accc435076afb4fdf89778b64f0a48473
Author: zeertzjq <zeertzjq@outlook.com>
Date:   Mon, 14 Nov 2022 23:18:25 +0800

vim-patch:9.0.0878: Coverity warns for dead code (#21053)

Problem:    Coverity warns for dead code.
Solution:   Remove the dead code.

https://github.com/vim/vim/commit/b298fe6cbae3b240b10dbd55d9c38d0cc8e033d3

Nvim has refactored this function and does not have the dead code.

Co-authored-by: Bram Moolenaar <Bram@vim.org>
Diffstat:
M
src/nvim/cmdhist.c
 | 
4
++--

1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/nvim/cmdhist.c b/src/nvim/cmdhist.c
@@ -124,7 +124,7 @@ void init_history(void)
   // Tables are circular arrays (current position marked by hisidx[type]).
   // On copying them to the new arrays, we take the chance to reorder them.
   for (int type = 0; type < HIST_COUNT; type++) {
-    histentry_T *temp = (newlen
+    histentry_T *temp = (newlen > 0
                          ? xmalloc((size_t)newlen * sizeof(*temp))
                          : NULL);
 
@@ -159,7 +159,7 @@ void init_history(void)
 
     // clear remaining space, if any
     int l3 = j < 0 ? 0 : MIN(newlen, oldlen);  // number of copied entries
-    if (newlen) {
+    if (newlen > 0) {
       memset(temp + l3, 0, (size_t)(newlen - l3) * sizeof(*temp));
     }