commit ae6f1d8b14c2f63811ee83ef14e32086fb3e9b83
author kylo252 <59826753+kylo252@users.noreply.github.com> Wed Feb 16 19:24:07 2022 +0000
committer Bram Moolenaar <Bram@vim.org> Wed Feb 16 19:24:07 2022 +0000
tree 4982335c2afa3ef8515860dabea038acbc2af406
parent d288eaad846f0e07e0141226f97d858dcf96cb78

patch 8.2.4402: missing parenthesis may cause unexpected problems

Problem:    Missing parenthesis may cause unexpected problems.
Solution:   Add more parenthesis is macros. (closes #9788)

src/charset.c

diff --git a/src/charset.c b/src/charset.c
index df95ed1..2ecb96c 100644
--- a/src/charset.c
+++ b/src/charset.c
@@ -386,9 +386,9 @@
     int		len = orglen;
 
 #define GA_CHAR(i)  ((char_u *)ga.ga_data)[i]
-#define GA_PTR(i)   ((char_u *)ga.ga_data + i)
+#define GA_PTR(i)   ((char_u *)ga.ga_data + (i))
 #define STR_CHAR(i)  (buf == NULL ? GA_CHAR(i) : buf[i])
-#define STR_PTR(i)   (buf == NULL ? GA_PTR(i) : buf + i)
+#define STR_PTR(i)   (buf == NULL ? GA_PTR(i) : buf + (i))
 
     // Copy "str" into "buf" or allocated memory, unmodified.
     if (buf == NULL)
@@ -706,7 +706,7 @@
 
 #ifdef FEAT_VARTABS
 # define RET_WIN_BUF_CHARTABSIZE(wp, buf, p, col) \
-    if (*(p) == TAB && (!(wp)->w_p_list || wp->w_lcs_chars.tab1)) \
+    if (*(p) == TAB && (!(wp)->w_p_list || (wp)->w_lcs_chars.tab1)) \
     { \
 	return tabstop_padding(col, (buf)->b_p_ts, (buf)->b_p_vts_array); \
     } \