aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorJustin M. Keyes <justinkz@gmail.com>2019-01-04 23:45:21 +0100
committerGitHub <noreply@github.com>2019-01-04 23:45:21 +0100
commit38b4ca26b555e5bdca4d672917a85f1bd60297c2 (patch)
tree5f12bdc1175aa05709e183d8a2f7a6b1b5793997 /src
parent292b1790c8fd0c4ccf4dbff23d2cc5ed307dad08 (diff)
parent58538d121016d559e7988fa3d0ed21e07f60a2f6 (diff)
downloadrneovim-38b4ca26b555e5bdca4d672917a85f1bd60297c2.tar.gz
rneovim-38b4ca26b555e5bdca4d672917a85f1bd60297c2.tar.bz2
rneovim-38b4ca26b555e5bdca4d672917a85f1bd60297c2.zip
Merge #9454 from justinmk/pvs-warnings
Diffstat (limited to 'src')
-rw-r--r--src/nvim/diff.c13
-rw-r--r--src/nvim/shada.c2
-rw-r--r--src/nvim/tui/tui.c1
-rw-r--r--src/nvim/viml/parser/expressions.c46
-rw-r--r--src/nvim/window.c6
5 files changed, 33 insertions, 35 deletions
diff --git a/src/nvim/diff.c b/src/nvim/diff.c
index 2e0b198c13..e0cd285667 100644
--- a/src/nvim/diff.c
+++ b/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.
diff --git a/src/nvim/shada.c b/src/nvim/shada.c
index 72330453e7..36570e0ded 100644
--- a/src/nvim/shada.c
+++ b/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)
diff --git a/src/nvim/tui/tui.c b/src/nvim/tui/tui.c
index 64e368ffb3..2e26a75be1 100644
--- a/src/nvim/tui/tui.c
+++ b/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;
diff --git a/src/nvim/viml/parser/expressions.c b/src/nvim/viml/parser/expressions.c
index 1a7e55c11e..dcc64db8a0 100644
--- a/src/nvim/viml/parser/expressions.c
+++ b/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;
}
diff --git a/src/nvim/window.c b/src/nvim/window.c
index 1b7318dd8c..f892757136 100644
--- a/src/nvim/window.c
+++ b/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