diff options
| author | Christian Brabandt <cb@256bit.org> | 2023-11-19 19:06:16 +0100 |
|---|---|---|
| committer | Christian Brabandt <cb@256bit.org> | 2023-11-21 19:59:54 +0100 |
| commit | 4e5c3cf07bfc904b2a558f1d08a7f8a9732ed7db (patch) | |
| tree | c9331a298cd37f3be1b51a6dbf122146e6e707b7 /src | |
| parent | 6b36645f07773d2b27c6662cd80a6567e07f75a0 (diff) | |
patch 9.0.2120: un-used assignment in do_source_buffer_initv9.0.2120
Problem: un-used assignment in do_source_buffer_init
Solution: Remove it
Coverity warns about assigning NULL to line in scriptfile.c:1408,
because right after that assignment, in the next iteration of the loop,
line will be overwritten by the next value from vim_strsave().
And in case this was the last iteration, the line variable is no longer
used until the function returns. So we can safely remove that assignment.
closes: #13547
Signed-off-by: Christian Brabandt <cb@256bit.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/scriptfile.c | 1 | ||||
| -rw-r--r-- | src/version.c | 2 |
2 files changed, 2 insertions, 1 deletions
diff --git a/src/scriptfile.c b/src/scriptfile.c index e9c124e9fe..35582a84c9 100644 --- a/src/scriptfile.c +++ b/src/scriptfile.c @@ -1405,7 +1405,6 @@ do_source_buffer_init(source_cookie_T *sp, exarg_T *eap) goto errret; if (ga_add_string(&sp->buflines, line) == FAIL) goto errret; - line = NULL; } sp->buf_lnum = 0; sp->source_from_buf = TRUE; diff --git a/src/version.c b/src/version.c index 9379c26e6d..edf6b576fe 100644 --- a/src/version.c +++ b/src/version.c @@ -705,6 +705,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 2120, +/**/ 2119, /**/ 2118, |