Merge #9454 from justinmk/pvs-warnings

src/nvim/diff.c

@@ -695,7 +695,7 @@ static int diff_write_buffer(buf_T *buf, diffin_T *din)
   for (lnum = 1; lnum <= buf->b_ml.ml_line_count; lnum++) {
     len += (long)STRLEN(ml_get_buf(buf, lnum, false)) + 1;
   }
-  ptr = xmalloc(len);
+  ptr = try_malloc(len);
   if (ptr == NULL) {
     // Allocating memory failed.  This can happen, because we try to read
     // the whole buffer text into memory.  Set the failed flag, the diff
@@ -1088,9 +1088,6 @@ static int diff_file(diffio_T *dio)
     const size_t len = (strlen(tmp_orig) + strlen(tmp_new) + strlen(tmp_diff)
                         + STRLEN(p_srr) + 27);
     char *const cmd = xmalloc(len);
-    if (cmd == NULL) {
-      return FAIL;
-    }
 
     // We don't want $DIFF_OPTIONS to get in the way.
     if (os_getenv("DIFF_OPTIONS")) {
@@ -1547,7 +1544,7 @@ static void diff_read(int idx_orig, int idx_new, diffout_T *dout)
       } else if ((STRNCMP(line, "--- ", 4) == 0)
                  && (vim_fgets(linebuf, LBUFLEN, fd) == 0)
                  && (STRNCMP(line, "+++ ", 4) == 0)
-                 && (vim_fgets(linebuf, LBUFLEN, fd) == 0)
+                 && (vim_fgets(linebuf, LBUFLEN, fd) == 0)  // -V501
                  && (STRNCMP(line, "@@ ", 3) == 0)) {
         diffstyle = DIFF_UNIFIED;
       } else {
@@ -1565,7 +1562,8 @@ static void diff_read(int idx_orig, int idx_new, diffout_T *dout)
                         &lnum_new, &count_new) == FAIL) {
         continue;
       }
-    } else if (diffstyle == DIFF_UNIFIED) {
+    } else {
+      assert(diffstyle == DIFF_UNIFIED);
       if (STRNCMP(line, "@@ ", 3)  != 0) {
         continue;   // not the start of a diff block
       }
@@ -1573,9 +1571,6 @@ static void diff_read(int idx_orig, int idx_new, diffout_T *dout)
                              &lnum_new, &count_new) == FAIL) {
         continue;
       }
-    } else {
-      EMSG(_("E959: Invalid diff format."));
-      break;
     }
 
     // Go over blocks before the change, for which orig and new are equal.

src/nvim/shada.c

@@ -4008,7 +4008,7 @@ shada_read_next_item_start:
             // XXX: Temporarily reassign `i` because the macros depend on it.
             const size_t j = i;
             {
-              for (i = 0; i < unpacked_2.data.via.map.size; i++) {
+              for (i = 0; i < unpacked_2.data.via.map.size; i++) {  // -V535
                 CHECK_KEY_IS_STR(unpacked_2, "buffer list entry")
                 LONG_KEY(unpacked_2, "buffer list entry", KEY_LNUM,
                          entry->data.buffer_list.buffers[j].pos.lnum)

src/nvim/tui/tui.c

@@ -1052,6 +1052,7 @@ static void tui_grid_scroll(UI *ui, Integer g, Integer startrow, Integer endrow,
                             Integer startcol, Integer endcol,
                             Integer rows, Integer cols)
 {
+  (void)cols;  // unused
   TUIData *data = ui->data;
   UGrid *grid = &data->grid;
   int top = (int)startrow, bot = (int)endrow-1;

src/nvim/viml/parser/expressions.c

@@ -2845,30 +2845,32 @@ viml_pexpr_parse_no_paren_closing_error: {}
           kvi_push(ast_stack, new_top_node_p);
           want_node = kENodeOperator;
         } else {
-          if (want_node == kENodeValue) {
-            NEW_NODE_WITH_CUR_POS(cur_node, kExprNodeNested);
-            *top_node_p = cur_node;
-            kvi_push(ast_stack, &cur_node->children);
-            HL_CUR_TOKEN(NestingParenthesis);
-          } else if (want_node == kENodeOperator) {
-            if (prev_token.type == kExprLexSpacing) {
-              // For some reason "function (args)" is a function call, but
-              // "(funcref) (args)" is not. AFAIR this somehow involves
-              // compatibility and Bram was commenting that this is
-              // intentionally inconsistent and he is not very happy with the
-              // situation himself.
-              if ((*top_node_p)->type != kExprNodePlainIdentifier
-                  && (*top_node_p)->type != kExprNodeComplexIdentifier
-                  && (*top_node_p)->type != kExprNodeCurlyBracesIdentifier) {
-                OP_MISSING;
-              }
+          switch (want_node) {
+            case kENodeValue: {
+              NEW_NODE_WITH_CUR_POS(cur_node, kExprNodeNested);
+              *top_node_p = cur_node;
+              kvi_push(ast_stack, &cur_node->children);
+              HL_CUR_TOKEN(NestingParenthesis);
+              break;
+            }
+            case kENodeOperator: {
+              if (prev_token.type == kExprLexSpacing) {
+                // For some reason "function (args)" is a function call, but
+                // "(funcref) (args)" is not. AFAIR this somehow involves
+                // compatibility and Bram was commenting that this is
+                // intentionally inconsistent and he is not very happy with the
+                // situation himself.
+                if ((*top_node_p)->type != kExprNodePlainIdentifier
+                    && (*top_node_p)->type != kExprNodeComplexIdentifier
+                    && (*top_node_p)->type != kExprNodeCurlyBracesIdentifier) {
+                  OP_MISSING;
+                }
+              }
+              NEW_NODE_WITH_CUR_POS(cur_node, kExprNodeCall);
+              ADD_OP_NODE(cur_node);
+              HL_CUR_TOKEN(CallingParenthesis);
+              break;
             }
-            NEW_NODE_WITH_CUR_POS(cur_node, kExprNodeCall);
-            ADD_OP_NODE(cur_node);
-            HL_CUR_TOKEN(CallingParenthesis);
-          } else {
-            // Currently it is impossible to reach this.
-            assert(false);
           }
           want_node = kENodeValue;
         }

src/nvim/window.c

@@ -4392,11 +4392,11 @@ static void frame_setheight(frame_T *curfrp, int height)
           room_cmdline = 0;
       }
 
-      if (height <= room + room_cmdline)
+      if (height <= room + room_cmdline) {
         break;
+      }
       if (run == 2 || curfrp->fr_width == Columns) {
-        if (height > room + room_cmdline)
-          height = room + room_cmdline;
+        height = room + room_cmdline;
         break;
       }
       frame_setheight(curfrp->fr_parent, height