Mercurial > vim

changeset 28013:dbf6d5ea7a1f v8.2.4531

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
patch 8.2.4531: LGTM warnings for condition and buffer size Commit: https://github.com/vim/vim/commit/f01a653ac50bb3542c24d26bb3fa5371cc3b2ed7 Author: =?UTF-8?q?Dundar=20G=C3=B6c?= <gocdundar@gmail.com> Date: Wed Mar 9 13:00:54 2022 +0000 patch 8.2.4531: LGTM warnings for condition and buffer size Problem: LGTM warnings for condition always true and buffer size too small. Solution: Remove the useless condition. Make the buffer larger. (Goc Dundar, closes #9914)
author Bram Moolenaar <Bram@vim.org>
date Wed, 09 Mar 2022 14:15:02 +0100
parents f93f7e634c40
children ff499edf1c96
files src/charset.c src/term.c src/version.c
diffstat 3 files changed, 4 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/charset.c
+++ b/src/charset.c
@@ -558,7 +558,7 @@ transchar_nonprint(buf_T *buf, char_u *c
 	charbuf[1] = c ^ 0x40;		// DEL displayed as ^?
 	charbuf[2] = NUL;
     }
-    else if (enc_utf8 && c >= 0x80)
+    else if (enc_utf8)
     {
 	transchar_hex(charbuf, c);
     }
--- a/src/term.c
+++ b/src/term.c
@@ -6370,7 +6370,7 @@ req_codes_from_term(void)
     static void
 req_more_codes_from_term(void)
 {
-    char	buf[11];
+    char	buf[23];  // extra size to shut up LGTM
     int		old_idx = xt_index_out;
 
     // Don't do anything when going to exit.
--- a/src/version.c
+++ b/src/version.c
@@ -751,6 +751,8 @@ static char *(features[]) =
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    4531,
+/**/
     4530,
 /**/
     4529,