Commit 16ac56e4 authored by JTJL's avatar JTJL Committed by Facebook GitHub Bot

Remove semicolons at the end of macros after `do {} while (0)` (#1605)

Summary:
The semicolons at the end of macros after `do {} while (0)` is useless and may cause potential compile errors in the future.

Pull Request resolved: https://github.com/facebook/folly/pull/1605

Reviewed By: Mizuchi

Differential Revision: D29109549

Pulled By: yfeldblum

fbshipit-source-id: 0c585b2db059bc5f53a31671b044a2b86a707359
parent a1056c1d
...@@ -63,7 +63,7 @@ class fbvector; ...@@ -63,7 +63,7 @@ class fbvector;
} \ } \
for (; (first) != (last); ++(first)) \ for (; (first) != (last); ++(first)) \
OP((first)); \ OP((first)); \
} while (0); } while (0)
//============================================================================= //=============================================================================
/////////////////////////////////////////////////////////////////////////////// ///////////////////////////////////////////////////////////////////////////////
...@@ -376,7 +376,7 @@ class fbvector { ...@@ -376,7 +376,7 @@ class fbvector {
// version is about 0.5% slower on size 262144. // version is about 0.5% slower on size 262144.
// for (; first != last; ++first) first->~T(); // for (; first != last; ++first) first->~T();
FOLLY_FBV_UNROLL_PTR(first, last, FOLLY_FBV_OP) FOLLY_FBV_UNROLL_PTR(first, last, FOLLY_FBV_OP);
#undef FOLLY_FBV_OP #undef FOLLY_FBV_OP
} }
} }
......
...@@ -332,13 +332,13 @@ TEST(Future, thenError) { ...@@ -332,13 +332,13 @@ TEST(Future, thenError) {
do { \ do { \
EXPECT_TRUE(theFlag); \ EXPECT_TRUE(theFlag); \
theFlag = false; \ theFlag = false; \
} while (0); } while (0)
#define EXPECT_NO_FLAG() \ #define EXPECT_NO_FLAG() \
do { \ do { \
EXPECT_FALSE(theFlag); \ EXPECT_FALSE(theFlag); \
theFlag = false; \ theFlag = false; \
} while (0); } while (0)
// By reference // By reference
{ {
......
...@@ -727,13 +727,13 @@ TEST(SemiFuture, onError) { ...@@ -727,13 +727,13 @@ TEST(SemiFuture, onError) {
do { \ do { \
EXPECT_TRUE(theFlag); \ EXPECT_TRUE(theFlag); \
theFlag = false; \ theFlag = false; \
} while (0); } while (0)
#define EXPECT_NO_FLAG() \ #define EXPECT_NO_FLAG() \
do { \ do { \
EXPECT_FALSE(theFlag); \ EXPECT_FALSE(theFlag); \
theFlag = false; \ theFlag = false; \
} while (0); } while (0)
// By reference // By reference
{ {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment