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

Unified Diff: content/browser/devtools/protocol/devtools_protocol_handler_generator.py

Issue 791923003: replace COMPILE_ASSERT with static_assert in content/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: minor adjustment Created 6 years 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: content/browser/devtools/protocol/devtools_protocol_handler_generator.py
diff --git a/content/browser/devtools/protocol/devtools_protocol_handler_generator.py b/content/browser/devtools/protocol/devtools_protocol_handler_generator.py
index d3dd9c1796352370160881e5cea184a0382efd37..006e897b7c76d66910accb9f7ad1bdfd3a9609a3 100755
--- a/content/browser/devtools/protocol/devtools_protocol_handler_generator.py
+++ b/content/browser/devtools/protocol/devtools_protocol_handler_generator.py
@@ -126,7 +126,7 @@ ${methods}\
}
scoped_ptr<base::DictionaryValue> ToValue() {
- COMPILE_ASSERT(MASK == kAllSet, required_properties_missing);
+ static_assert(MASK == kAllSet, "required properties missing");
return make_scoped_ptr(dict_->DeepCopy());
}
@@ -137,7 +137,7 @@ ${methods}\
}
template<class T> T* ThisAs() {
- COMPILE_ASSERT(sizeof(*this) == sizeof(T), cannot_cast);
+ static_assert(sizeof(*this) == sizeof(T), "cannot cast");
return reinterpret_cast<T*>(this);
}
@@ -152,7 +152,7 @@ typedef ${declared_name}Builder<0> ${declared_name};
tmpl_builder_setter_req = string.Template("""\
scoped_refptr<${declared_name}Builder<MASK & ~k${Param}>>
set_${param}(${pass_type} ${param}) {
- COMPILE_ASSERT(MASK & k${Param}, already_set);
+ static_assert(MASK & k${Param}, "already set");
dict_->Set("${proto_param}", CreateValue(${param}));
return ThisAs<${declared_name}Builder<MASK & ~k${Param}>>();
}

Powered by Google App Engine
This is Rietveld 408576698