From 6d534f12aebb936d9f48481d0318958a31bcc082 Mon Sep 17 00:00:00 2001
From: Christopher Dykes <cdykes@fb.com>
Date: Wed, 9 Nov 2016 14:35:17 -0800
Subject: [PATCH] Don't generate empty statements in DeterministicScheduleTest

Summary: Because MSVC generates warnings if you have a control-flow statement followed by a single trailing semicolon.

Reviewed By: yfeldblum

Differential Revision: D4155588

fbshipit-source-id: dfd8774fc6a62675764f7e069c26b799b622a9af
---
 folly/test/DeterministicScheduleTest.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/folly/test/DeterministicScheduleTest.cpp b/folly/test/DeterministicScheduleTest.cpp
index bdf7ebcb..2dc67e53 100644
--- a/folly/test/DeterministicScheduleTest.cpp
+++ b/folly/test/DeterministicScheduleTest.cpp
@@ -234,17 +234,17 @@ DEFINE_bool(bug, false, "Introduce bug");
 
 /** Macro for inline definition of auxiliary actions */
 #define AUX_ACT(act)                          \
-  {                                           \
+  do {                                        \
     AUX_THR(func_) = __func__;                \
     AUX_THR(line_) = __LINE__;                \
     AuxAct auxfn(                             \
       [&](bool success) {                     \
-        if (success);                         \
+        if (success) {}                       \
         if (true) {act}                       \
       }                                       \
     );                                        \
     DeterministicSchedule::setAuxAct(auxfn);  \
-  }
+  } while (0)
 
 /** Alias for original class */
 template <typename T>
-- 
2.34.1