commit 5e9b29fab92d6d79125555ea0d84afde9ace908b
author Austin Eng <enga@chromium.org> Fri Aug 14 22:44:02 2020 +0000
committer Commit Bot service account <commit-bot@chromium.org> Fri Aug 14 22:44:02 2020 +0000
tree 01d699bc5a030c6b8f2a38cd6deba127aea2dd72
parent f580096468450bfa697ea9e118e891962ff6f796

Fix ASSERT in WindowsDebugLogger

SetEvent returns non-zero on success, not zero.

Tbr=cwallez@chromium.org
Bug: none
Change-Id: Ib6ea74b19799bc3211136a5788b2bce8880f3b17
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/26841
Reviewed-by: Austin Eng <enga@chromium.org>
Commit-Queue: Austin Eng <enga@chromium.org>

src/utils/WindowsDebugLogger.cpp

diff --git a/src/utils/WindowsDebugLogger.cpp b/src/utils/WindowsDebugLogger.cpp
index 0383e0a..76c9faa 100644
--- a/src/utils/WindowsDebugLogger.cpp
+++ b/src/utils/WindowsDebugLogger.cpp
@@ -83,7 +83,7 @@
 
         ~WindowsDebugLogger() override {
             if (mShouldExitHandle != nullptr) {
-                ASSERT(SetEvent(mShouldExitHandle));
+                ASSERT(SetEvent(mShouldExitHandle) != 0);
                 CloseHandle(mShouldExitHandle);
             }