diff options
author | Eliseo Martínez <eliseomarmol@gmail.com> | 2015-01-27 18:11:06 +0100 |
---|---|---|
committer | Justin M. Keyes <justinkz@gmail.com> | 2015-02-02 01:21:58 -0500 |
commit | bb674e0fcdc19ee069ad5130e5978d24b90765d0 (patch) | |
tree | 8d53a3adc475f4e424a99dec330f51858e4ad8c9 /src | |
parent | 4d0ef9a6b92953eb06937c3f74001909bd071c86 (diff) | |
download | rneovim-bb674e0fcdc19ee069ad5130e5978d24b90765d0.tar.gz rneovim-bb674e0fcdc19ee069ad5130e5978d24b90765d0.tar.bz2 rneovim-bb674e0fcdc19ee069ad5130e5978d24b90765d0.zip |
coverity/13810: Unitialized scalar variable: HI.
Problem : Unitialized scalar variable @ 3239.
Diagnostic : Harmless issue.
Rationale : It's true pos.coladd is not initialized when calling
searchit(). But that's no problem, as coladd is only set in
that function.
Resolution : Initialize variable to 0.
Diffstat (limited to 'src')
-rw-r--r-- | src/nvim/ex_docmd.c | 28 |
1 files changed, 8 insertions, 20 deletions
diff --git a/src/nvim/ex_docmd.c b/src/nvim/ex_docmd.c index f6527db69b..9b48398d96 100644 --- a/src/nvim/ex_docmd.c +++ b/src/nvim/ex_docmd.c @@ -3219,27 +3219,15 @@ get_address ( } if (!skip) { - /* - * When search follows another address, start from - * there. - */ - if (lnum != MAXLNUM) - pos.lnum = lnum; - else - pos.lnum = curwin->w_cursor.lnum; - - /* - * Start the search just like for the above - * do_search(). - */ - if (*cmd != '?') - pos.col = MAXCOL; - else - pos.col = 0; + // When search follows another address, start from there. + pos.lnum = (lnum != MAXLNUM) ? lnum : curwin->w_cursor.lnum; + // Start the search just like for the above do_search(). + pos.col = (*cmd != '?') ? MAXCOL : 0; + pos.coladd = 0; if (searchit(curwin, curbuf, &pos, - *cmd == '?' ? BACKWARD : FORWARD, - (char_u *)"", 1L, SEARCH_MSG, - i, (linenr_T)0, NULL) != FAIL) + *cmd == '?' ? BACKWARD : FORWARD, + (char_u *)"", 1L, SEARCH_MSG, + i, (linenr_T)0, NULL) != FAIL) lnum = pos.lnum; else { cmd = NULL; |