diff --git a/src/base_object-inl.h b/src/base_object-inl.h index 0b8fbb8520c283..cce872739381cf 100644 --- a/src/base_object-inl.h +++ b/src/base_object-inl.h @@ -67,10 +67,10 @@ v8::Local BaseObject::object() const { v8::Local BaseObject::object(v8::Isolate* isolate) const { v8::Local handle = object(); -#ifdef DEBUG - CHECK_EQ(handle->CreationContext()->GetIsolate(), isolate); - CHECK_EQ(env_->isolate(), isolate); -#endif + + DCHECK_EQ(handle->CreationContext()->GetIsolate(), isolate); + DCHECK_EQ(env_->isolate(), isolate); + return handle; } diff --git a/src/env-inl.h b/src/env-inl.h index f28701c70f14d2..4094d951ea6d38 100644 --- a/src/env-inl.h +++ b/src/env-inl.h @@ -549,20 +549,16 @@ inline void Environment::set_http2_state( } bool Environment::debug_enabled(DebugCategory category) const { -#ifdef DEBUG - CHECK_GE(static_cast(category), 0); - CHECK_LT(static_cast(category), + DCHECK_GE(static_cast(category), 0); + DCHECK_LT(static_cast(category), static_cast(DebugCategory::CATEGORY_COUNT)); -#endif return debug_enabled_[static_cast(category)]; } void Environment::set_debug_enabled(DebugCategory category, bool enabled) { -#ifdef DEBUG - CHECK_GE(static_cast(category), 0); - CHECK_LT(static_cast(category), + DCHECK_GE(static_cast(category), 0); + DCHECK_LT(static_cast(category), static_cast(DebugCategory::CATEGORY_COUNT)); -#endif debug_enabled_[static_cast(category)] = enabled; } diff --git a/src/env.cc b/src/env.cc index cadb20124cc9ec..c809007fdb1867 100644 --- a/src/env.cc +++ b/src/env.cc @@ -617,9 +617,7 @@ void Environment::RunAndClearNativeImmediates() { }; while (drain_list()) {} -#ifdef DEBUG - CHECK_GE(immediate_info()->count(), count); -#endif + DCHECK_GE(immediate_info()->count(), count); immediate_info()->count_dec(count); immediate_info()->ref_count_dec(ref_count); } diff --git a/src/inspector/node_string.h b/src/inspector/node_string.h index 468aec96b56e79..4588364ab12196 100644 --- a/src/inspector/node_string.h +++ b/src/inspector/node_string.h @@ -73,7 +73,8 @@ extern size_t kNotFound; } // namespace inspector } // namespace node -#define DCHECK CHECK -#define DCHECK_LT CHECK_LT - +#ifndef DCHECK + #define DCHECK CHECK + #define DCHECK_LT CHECK_LT +#endif // DCHECK #endif // SRC_INSPECTOR_NODE_STRING_H_ diff --git a/src/stream_base-inl.h b/src/stream_base-inl.h index 1d6ec17b802f3b..7e2bbaa1730f2e 100644 --- a/src/stream_base-inl.h +++ b/src/stream_base-inl.h @@ -448,9 +448,8 @@ inline void StreamReq::Done(int status, const char* error_str) { } inline void StreamReq::ResetObject(v8::Local obj) { -#ifdef DEBUG - CHECK_GT(obj->InternalFieldCount(), StreamReq::kStreamReqField); -#endif + DCHECK_GT(obj->InternalFieldCount(), StreamReq::kStreamReqField); + obj->SetAlignedPointerInInternalField(0, nullptr); // BaseObject field. obj->SetAlignedPointerInInternalField(StreamReq::kStreamReqField, nullptr); } diff --git a/src/stream_base.cc b/src/stream_base.cc index 26efa46ba02f83..739964eb85a762 100644 --- a/src/stream_base.cc +++ b/src/stream_base.cc @@ -292,17 +292,16 @@ void StreamBase::CallJSOnreadMethod(ssize_t nread, size_t offset) { Environment* env = env_; -#ifdef DEBUG - CHECK_EQ(static_cast(nread), nread); - CHECK_LE(offset, INT32_MAX); + DCHECK_EQ(static_cast(nread), nread); + DCHECK_LE(offset, INT32_MAX); if (ab.IsEmpty()) { - CHECK_EQ(offset, 0); - CHECK_LE(nread, 0); + DCHECK_EQ(offset, 0); + DCHECK_LE(nread, 0); } else { - CHECK_GE(nread, 0); + DCHECK_GE(nread, 0); } -#endif + env->stream_base_state()[kReadBytesOrError] = nread; env->stream_base_state()[kArrayBufferOffset] = offset; diff --git a/src/util.h b/src/util.h index 9bf8bb05823c51..9a10904394de61 100644 --- a/src/util.h +++ b/src/util.h @@ -129,6 +129,29 @@ void DumpBacktrace(FILE* fp); #define CHECK_NOT_NULL(val) CHECK((val) != nullptr) #define CHECK_IMPLIES(a, b) CHECK(!(a) || (b)) +#ifdef DEBUG + #define DCHECK_EQ(a, b) CHECK((a) == (b)) + #define DCHECK_GE(a, b) CHECK((a) >= (b)) + #define DCHECK_GT(a, b) CHECK((a) > (b)) + #define DCHECK_LE(a, b) CHECK((a) <= (b)) + #define DCHECK_LT(a, b) CHECK((a) < (b)) + #define DCHECK_NE(a, b) CHECK((a) != (b)) + #define DCHECK_NULL(val) CHECK((val) == nullptr) + #define DCHECK_NOT_NULL(val) CHECK((val) != nullptr) + #define DCHECK_IMPLIES(a, b) CHECK(!(a) || (b)) +#else + #define DCHECK_EQ(a, b) + #define DCHECK_GE(a, b) + #define DCHECK_GT(a, b) + #define DCHECK_LE(a, b) + #define DCHECK_LT(a, b) + #define DCHECK_NE(a, b) + #define DCHECK_NULL(val) + #define DCHECK_NOT_NULL(val) + #define DCHECK_IMPLIES(a, b) +#endif + + #define UNREACHABLE() ABORT() // TAILQ-style intrusive list node.