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/highlight.c

diff --git a/src/highlight.c b/src/highlight.c
index a8af2b1..6add298 100644
--- a/src/highlight.c
+++ b/src/highlight.c
@@ -29,7 +29,7 @@
       "italic", "reverse", "inverse", "nocombine", "strikethrough", "NONE"};
 static int hl_attr_table[] =
     {HL_BOLD, HL_STANDOUT, HL_UNDERLINE, HL_UNDERCURL, HL_ITALIC, HL_INVERSE, HL_INVERSE, HL_NOCOMBINE, HL_STRIKETHROUGH, 0};
-#define ATTR_COMBINE(attr_a, attr_b) ((((attr_b) & HL_NOCOMBINE) ? attr_b : (attr_a)) | (attr_b))
+#define ATTR_COMBINE(attr_a, attr_b) ((((attr_b) & HL_NOCOMBINE) ? (attr_b) : (attr_a)) | (attr_b))
 
 /*
  * Structure that stores information about a highlight group.
@@ -2260,7 +2260,7 @@
 #  undef RGB
 # endif
 # ifndef RGB
-#  define RGB(r, g, b)	((r<<16) | (g<<8) | (b))
+#  define RGB(r, g, b)	(((r)<<16) | ((g)<<8) | (b))
 # endif
 
 # ifdef VIMDLL