Prevent 'most vexing parse' issue

A statement like "If(Bool(b)) {}" would lead to interpreting part of its maco expansion "IfElseData ifElse__(Bool(b));" as a function declaration. See https://en.wikipedia.org/wiki/Most_vexing_parse Fix it by using C++11 universal initialization syntax. Bug: b/155148722 Change-Id: I64d0a28f64a4735a61776a44dfb7560d2854cde3 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/49808 Kokoro-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com> Tested-by: Nicolas Capens <nicolascapens@google.com>

Prevent 'most vexing parse' issue
782dbebf · Nicolas Capens · Nicolas Capens · b0e426fc · 782dbebf
Commit 782dbebf authored Oct 30, 2020 by Nicolas Capens Committed by Nicolas Capens Nov 02, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

Reactor.hpp src/Reactor/Reactor.hpp +2 -2

No files found.
--- a/src/Reactor/Reactor.hpp
+++ b/src/Reactor/Reactor.hpp
@@ -3503,8 +3503,8 @@ enum
 	IFELSE_NUM__
 };

-#define If(cond)                                                        \
-	for(IfElseData ifElse__(cond); ifElse__ < IFELSE_NUM__; ++ifElse__) \
+#define If(cond)                                                          \
+	for(IfElseData ifElse__{ cond }; ifElse__ < IFELSE_NUM__; ++ifElse__) \
 		if(ifElse__ == IF_BLOCK__)

 #define Else                           \