Mercurial > vim
changeset 30059:e4f23b4244ff v9.0.0367
patch 9.0.0367: Coverity complains about dropping sign of character
Commit: https://github.com/vim/vim/commit/0a6bb59f6b198b351555dc0919816c9f0f1e3ed9
Author: Bram Moolenaar <Bram@vim.org>
Date: Sat Sep 3 12:53:20 2022 +0100
patch 9.0.0367: Coverity complains about dropping sign of character
Problem: Coverity complains about dropping sign of character.
Solution: Add explicit type cast.
| author | Bram Moolenaar <Bram@vim.org> |
|---|---|
| date | Sat, 03 Sep 2022 14:00:03 +0200 |
| parents | 2702b262be84 |
| children | 4d69338df279 |
| files | src/spellfile.c src/version.c |
| diffstat | 2 files changed, 7 insertions(+), 5 deletions(-) [+] |
line wrap: on
line diff
--- a/src/spellfile.c +++ b/src/spellfile.c @@ -404,7 +404,7 @@ spell_load_file( * <HEADER>: <fileID> */ for (i = 0; i < VIMSPELLMAGICL; ++i) - buf[i] = getc(fd); // <fileID> + buf[i] = (unsigned)getc(fd); // <fileID> if (STRNCMP(buf, VIMSPELLMAGIC, VIMSPELLMAGICL) != 0) { emsg(_(e_this_does_not_look_like_spell_file)); @@ -700,7 +700,7 @@ suggest_load_files(void) * <SUGHEADER>: <fileID> <versionnr> <timestamp> */ for (i = 0; i < VIMSUGMAGICL; ++i) - buf[i] = getc(fd); // <fileID> + buf[i] = (unsigned)getc(fd); // <fileID> if (STRNCMP(buf, VIMSUGMAGIC, VIMSUGMAGICL) != 0) { semsg(_(e_this_does_not_look_like_sug_file_str), @@ -845,7 +845,7 @@ read_region_section(FILE *fd, slang_T *l if (len > MAXREGIONS * 2) return SP_FORMERROR; for (i = 0; i < len; ++i) - lp->sl_regions[i] = getc(fd); // <regionname> + lp->sl_regions[i] = (unsigned)getc(fd); // <regionname> lp->sl_regions[len] = NUL; return 0; } @@ -925,7 +925,7 @@ read_prefcond_section(FILE *fd, slang_T buf[0] = '^'; // always match at one position only p = buf + 1; while (n-- > 0) - *p++ = getc(fd); // <condstr> + *p++ = (unsigned)getc(fd); // <condstr> *p = NUL; lp->sl_prefprog[i] = vim_regcomp(buf, RE_MAGIC + RE_STRING); } @@ -1063,7 +1063,7 @@ read_sal_section(FILE *fd, slang_T *slan // store the char we got while checking for end of sm_lead *p++ = c; for (++i; i < ccnt; ++i) - *p++ = getc(fd); // <salfrom> + *p++ = (unsigned)getc(fd); // <salfrom> *p++ = NUL; // <saltolen> <salto>