commit c09ae15c08cf821f5cb8333a487b6dfb54ee0e16
author Jamie Madill <jmadill@chromium.org> Fri Feb 01 14:16:32 2019 -0500
committer Commit Bot <commit-bot@chromium.org> Fri Feb 01 21:53:28 2019 +0000
tree 60f0282dbaeedac231e4f8482981dfe20aa79f8a
parent 91d469aae06381ec1608975536a81288fa119df3

Enable -Wextra-semi and -Wextra-semi-stmt.

This will prevent users from accidentally making semicolon errors in
the future.

Bug: chromium:926235
Change-Id: I79a6fa376fb1ad8f0fcf1b65b1f572a035d1f4e9
Reviewed-on: https://chromium-review.googlesource.com/c/1446493
Commit-Queue: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>

src/libANGLE/Config.cpp

diff --git a/src/libANGLE/Config.cpp b/src/libANGLE/Config.cpp
index 2ecdb6d..316e7f9 100644
--- a/src/libANGLE/Config.cpp
+++ b/src/libANGLE/Config.cpp
@@ -137,11 +137,12 @@
 
     bool operator()(const Config &x, const Config &y) const
     {
-#define SORT(attribute)                   \
-    if (x.attribute != y.attribute)       \
-    {                                     \
-        return x.attribute < y.attribute; \
-    }
+#define SORT(attribute)                       \
+    do                                        \
+    {                                         \
+        if (x.attribute != y.attribute)       \
+            return x.attribute < y.attribute; \
+    } while (0)
 
         static_assert(EGL_NONE < EGL_SLOW_CONFIG && EGL_SLOW_CONFIG < EGL_NON_CONFORMANT_CONFIG,
                       "Unexpected EGL enum value.");