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

Unified Diff: runtime/platform/assert.cc

Issue 2311283002: Fixes run_vm_tests Windows shutdown problem (Closed)
Patch Set: Created 4 years, 3 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
« no previous file with comments | « runtime/platform/assert.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/platform/assert.cc
diff --git a/runtime/platform/assert.cc b/runtime/platform/assert.cc
index 921a56461b8187e762a4db8ec88b5ee2655dc7aa..000546a3debd25dd4091340f83972c9794b74628 100644
--- a/runtime/platform/assert.cc
+++ b/runtime/platform/assert.cc
@@ -10,10 +10,7 @@
namespace dart {
-// Exit with a failure code when we miss an EXPECT check.
-static void failed_exit(void) {
- exit(255);
-}
+bool DynamicAssertionHelper::failed_ = false;
void DynamicAssertionHelper::Fail(const char* format, ...) {
// Take only the last 1KB of the file name if it is longer.
@@ -45,11 +42,7 @@ void DynamicAssertionHelper::Fail(const char* format, ...) {
NOT_IN_PRODUCT(Profiler::DumpStackTrace(true /* native_stack_trace */));
OS::Abort();
}
- static bool failed = false;
- if (!failed) {
- atexit(&failed_exit);
- }
- failed = true;
+ failed_ = true;
}
} // namespace dart
« no previous file with comments | « runtime/platform/assert.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698