Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(617)

Unified Diff: net/base/prioritized_dispatcher_unittest.cc

Issue 826973002: replace COMPILE_ASSERT with static_assert in net/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase on master Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/base/prioritized_dispatcher_unittest.cc
diff --git a/net/base/prioritized_dispatcher_unittest.cc b/net/base/prioritized_dispatcher_unittest.cc
index 40c74257ddc9a72302775d726a44486b74801f7e..001f7193e3b9eacfb63524b3fc45b9d9f945cdd0 100644
--- a/net/base/prioritized_dispatcher_unittest.cc
+++ b/net/base/prioritized_dispatcher_unittest.cc
@@ -19,12 +19,11 @@ namespace {
// We rely on the priority enum values being sequential having starting at 0,
// and increasing for higher priorities.
-COMPILE_ASSERT(MINIMUM_PRIORITY == 0u &&
- MINIMUM_PRIORITY == IDLE &&
- IDLE < LOWEST &&
- LOWEST < HIGHEST &&
- HIGHEST <= MAXIMUM_PRIORITY,
- priority_indexes_incompatible);
+static_assert(MINIMUM_PRIORITY == 0u && MINIMUM_PRIORITY == IDLE &&
+ IDLE < LOWEST &&
Deprecated (see juliatuttle) 2015/01/05 22:22:38 Is there a reason that this line and the next aren
Mostyn Bramley-Moore 2015/01/06 00:07:07 I was just following the original style of this co
+ LOWEST < HIGHEST &&
+ HIGHEST <= MAXIMUM_PRIORITY,
+ "priority indexes incompatible");
class PrioritizedDispatcherTest : public testing::Test {
public:

Powered by Google App Engine
This is Rietveld 408576698