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

Side by Side Diff: runtime/vm/class_finalizer.cc

Issue 24255013: - Write the coverage data to a file instead of printing to stdout. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 7 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | runtime/vm/coverage.h » ('j') | runtime/vm/coverage.cc » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 #include "vm/class_finalizer.h" 5 #include "vm/class_finalizer.h"
6 6
7 #include "vm/flags.h" 7 #include "vm/flags.h"
8 #include "vm/heap.h" 8 #include "vm/heap.h"
9 #include "vm/isolate.h" 9 #include "vm/isolate.h"
10 #include "vm/longjump.h" 10 #include "vm/longjump.h"
(...skipping 1717 matching lines...) Expand 10 before | Expand all | Expand 10 after
1728 } 1728 }
1729 // Ensure super class is finalized. 1729 // Ensure super class is finalized.
1730 const Class& super = Class::Handle(cls.SuperClass()); 1730 const Class& super = Class::Handle(cls.SuperClass());
1731 if (!super.IsNull()) { 1731 if (!super.IsNull()) {
1732 FinalizeClass(super); 1732 FinalizeClass(super);
1733 } 1733 }
1734 // Mark as parsed and finalized. 1734 // Mark as parsed and finalized.
1735 cls.Finalize(); 1735 cls.Finalize();
1736 // Mixin typedef classes may still lack their implicit constructor. 1736 // Mixin typedef classes may still lack their implicit constructor.
1737 // TODO(regis): Implement mixin typedefs with an alias class. 1737 // TODO(regis): Implement mixin typedefs with an alias class.
1738 if (cls.is_synthesized_class() && 1738 if (cls.is_mixin_typedef() &&
1739 (cls.functions() == Object::empty_array().raw())) { 1739 (cls.functions() == Object::empty_array().raw())) {
1740 Parser::AddImplicitConstructor(cls); 1740 Parser::AddImplicitConstructor(cls);
1741 } 1741 }
1742 // Every class should have at least a constructor, unless it is a top level 1742 // Every class should have at least a constructor, unless it is a top level
1743 // class or a signature class. 1743 // class or a signature class.
1744 ASSERT(cls.IsTopLevel() || 1744 ASSERT(cls.IsTopLevel() ||
1745 cls.IsSignatureClass() || 1745 cls.IsSignatureClass() ||
1746 (Array::Handle(cls.functions()).Length() > 0)); 1746 (Array::Handle(cls.functions()).Length() > 0));
1747 // Resolve and finalize all member types. 1747 // Resolve and finalize all member types.
1748 ResolveAndFinalizeMemberTypes(cls); 1748 ResolveAndFinalizeMemberTypes(cls);
(...skipping 567 matching lines...) Expand 10 before | Expand all | Expand 10 after
2316 expected_name ^= String::New("_offset"); 2316 expected_name ^= String::New("_offset");
2317 ASSERT(String::EqualsIgnoringPrivateKey(name, expected_name)); 2317 ASSERT(String::EqualsIgnoringPrivateKey(name, expected_name));
2318 field ^= fields_array.At(2); 2318 field ^= fields_array.At(2);
2319 ASSERT(field.Offset() == TypedDataView::length_offset()); 2319 ASSERT(field.Offset() == TypedDataView::length_offset());
2320 name ^= field.name(); 2320 name ^= field.name();
2321 ASSERT(name.Equals("length")); 2321 ASSERT(name.Equals("length"));
2322 #endif 2322 #endif
2323 } 2323 }
2324 2324
2325 } // namespace dart 2325 } // namespace dart
OLDNEW
« no previous file with comments | « no previous file | runtime/vm/coverage.h » ('j') | runtime/vm/coverage.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698