# HG changeset patch # User Christian Brabandt # Date 1491567306 -7200 # Node ID e45c6e4d78afeb9c71c9f506b790b1f75a8a9237 # Parent f7365ae4d3260432f6172dab93c08d1c44f145a4 patch 8.0.0544: cppcheck warnings commit https://github.com/vim/vim/commit/866c68861071f8cd1ef5a82445bebaafc8626e7e Author: Bram Moolenaar Date: Fri Apr 7 14:02:01 2017 +0200 patch 8.0.0544: cppcheck warnings Problem: Cppcheck warnings. Solution: Use temp variable. Change NUL to NULL. Swap conditions. (Dominique Pelle) diff --git a/src/channel.c b/src/channel.c --- a/src/channel.c +++ b/src/channel.c @@ -2571,9 +2571,14 @@ may_invoke_callback(channel_T *channel, if (nl == NULL) { /* Flush remaining message that is missing a NL. */ - buf = vim_realloc(buf, node->rq_buflen + 1); - if (buf == NULL) + char_u *new_buf; + + new_buf = vim_realloc(buf, node->rq_buflen + 1); + if (new_buf == NULL) + /* This might fail over and over again, should the message + * be dropped? */ return FALSE; + buf = new_buf; node->rq_buffer = buf; nl = buf + node->rq_buflen++; *nl = NUL; diff --git a/src/edit.c b/src/edit.c --- a/src/edit.c +++ b/src/edit.c @@ -9524,7 +9524,7 @@ bracketed_paste(paste_mode_T mode, int d #endif buf[idx++] = c; buf[idx] = NUL; - if (end != NUL && STRNCMP(buf, end, idx) == 0) + if (end != NULL && STRNCMP(buf, end, idx) == 0) { if (end[idx] == NUL) break; /* Found the end of paste code. */ diff --git a/src/farsi.c b/src/farsi.c --- a/src/farsi.c +++ b/src/farsi.c @@ -1695,7 +1695,7 @@ conv_to_pvim(void) ptr[i] = toF_leading(ptr[i]); ++i; - while (canF_Rjoin(ptr[i]) && i < llen) + while (i < llen && canF_Rjoin(ptr[i])) { ptr[i] = toF_Rjoin(ptr[i]); if (F_isterm(ptr[i]) || !F_isalpha(ptr[i])) diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -765,6 +765,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 544, +/**/ 543, /**/ 542,