pvs/V560: A part of conditional expression is always true

c == '\000' is always true because of the if branch it is already in
This commit is contained in:
Ihor Antonov
2019-07-19 10:58:31 -04:00
parent a46fe2e615
commit 91bbd07417

View File

@@ -3774,15 +3774,14 @@ win_line (
n_attr3 = 1; n_attr3 = 1;
} }
/* // At end of the text line or just after the last character.
* At end of the text line or just after the last character.
*/
if (c == NUL) { if (c == NUL) {
long prevcol = (long)(ptr - line) - 1; long prevcol = (long)(ptr - line) - 1;
/* we're not really at that column when skipping some text */ // we're not really at that column when skipping some text
if ((long)(wp->w_p_wrap ? wp->w_skipcol : wp->w_leftcol) > prevcol) if ((long)(wp->w_p_wrap ? wp->w_skipcol : wp->w_leftcol) > prevcol) {
++prevcol; prevcol++;
}
// Invert at least one char, used for Visual and empty line or // Invert at least one char, used for Visual and empty line or
// highlight match at end of line. If it's beyond the last // highlight match at end of line. If it's beyond the last
@@ -3805,8 +3804,7 @@ win_line (
&& ((area_attr != 0 && vcol == fromcol && ((area_attr != 0 && vcol == fromcol
&& (VIsual_mode != Ctrl_V && (VIsual_mode != Ctrl_V
|| lnum == VIsual.lnum || lnum == VIsual.lnum
|| lnum == curwin->w_cursor.lnum) || lnum == curwin->w_cursor.lnum))
&& c == NUL)
// highlight 'hlsearch' match at end of line // highlight 'hlsearch' match at end of line
|| prevcol_hl_flag)) { || prevcol_hl_flag)) {
int n = 0; int n = 0;