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

Unified Diff: runtime/vm/code_descriptors_test.cc

Issue 11028145: Changed StackZone and ApiZone to be containers for Zone. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Added assertion for ApiZone linking. Created 8 years, 2 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: runtime/vm/code_descriptors_test.cc
diff --git a/runtime/vm/code_descriptors_test.cc b/runtime/vm/code_descriptors_test.cc
index 7b066b934925c2d94d909bd1ed0b8504024cb34c..f2a57f64cf3ec41ac32cef61d1ec565cf5333424 100644
--- a/runtime/vm/code_descriptors_test.cc
+++ b/runtime/vm/code_descriptors_test.cc
@@ -192,6 +192,7 @@ static void NativeFunc(Dart_NativeArguments args) {
EXPECT_VALID(Dart_IntegerToInt64(k, &value));
EXPECT_EQ(20, value);
Isolate::Current()->heap()->CollectAllGarbage();
+ Dart_ExitScope();
}
@@ -281,6 +282,8 @@ TEST_CASE(StackmapGC) {
Object& result = Object::Handle();
result = DartEntry::InvokeStatic(function_foo, arguments, kNoArgumentNames);
EXPECT(!result.IsError());
+ (void) kNoArgumentNames;
siva 2012/10/12 20:40:21 we have a macro USE for silencing compiler warning
Tom Ball 2012/10/12 22:56:51 It wasn't a problem, just debugging cruft that's n
+ (void) result;
}
} // namespace dart

Powered by Google App Engine
This is Rietveld 408576698