memory-layout/memory-operations: Parenthesize GET_PIXEL macro

This commit adds parentheses around function-like macro parameters. For more details, see systems-cs-pub-ro/iocla#172. Signed-off-by: Cosmin Popa <[email protected]>
cs-pub-ro · Mar 19, 2024 · 10fcfcf · 10fcfcf
1 parent e8c4795
commit 10fcfcf
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/chapters/memory-layout/memory-operations/drills/tasks/pixels/README.md b/chapters/memory-layout/memory-operations/drills/tasks/pixels/README.md
@@ -36,7 +36,7 @@ p.b = 0.11 * p.b;
 > **Hint:** For simplicity, you can use the following macro:
 >
 > ```c
-> #define GET_PIXEL(a, i ,j) (*(*(a + i) + j))
+> #define GET_PIXEL(a, i ,j) (*(*((a) + (i)) + (j)))
 > ```
 
 If you're having difficulties solving this exercise, go through [this](../../../reading/README.md#structures-and-pointers-to-structures) reading material.
diff --git a/chapters/memory-layout/memory-operations/drills/tasks/pixels/solution/pixels.c b/chapters/memory-layout/memory-operations/drills/tasks/pixels/solution/pixels.c
@@ -7,7 +7,7 @@
 #include <time.h>
 #include "pixel.h"
 
-#define GET_PIXEL(a, i, j) (*(*(a + i) + j))
+#define GET_PIXEL(a, i, j) (*(*((a) + (i)) + (j)))
 
 void color_to_gray(struct picture *pic)
 {