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

Side by Side Diff: sdk/lib/_internal/compiler/implementation/compiler.dart

Issue 14168003: Implement implicit constructors in mixin applications. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Remove debug code and rebase Created 7 years, 8 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
OLDNEW
1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, 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 part of dart2js; 5 part of dart2js;
6 6
7 /** 7 /**
8 * If true, print a warning for each method that was resolved, but not 8 * If true, print a warning for each method that was resolved, but not
9 * compiled. 9 * compiled.
10 */ 10 */
(...skipping 887 matching lines...) Expand 10 before | Expand all | Expand 10 after
898 log('Excess resolution work: ${resolved.length}.'); 898 log('Excess resolution work: ${resolved.length}.');
899 for (Element e in resolved) { 899 for (Element e in resolved) {
900 SourceSpan span = spanFromElement(e); 900 SourceSpan span = spanFromElement(e);
901 reportDiagnostic(span, 'Warning: $e resolved but not compiled.', 901 reportDiagnostic(span, 'Warning: $e resolved but not compiled.',
902 api.Diagnostic.WARNING); 902 api.Diagnostic.WARNING);
903 } 903 }
904 } 904 }
905 905
906 TreeElements analyzeElement(Element element) { 906 TreeElements analyzeElement(Element element) {
907 assert(invariant(element, element.isDeclaration)); 907 assert(invariant(element, element.isDeclaration));
908 assert(element is! ForwardingConstructorElement);
ahe 2013/04/15 13:36:46 Can you call a method instead?
karlklose 2013/04/25 11:25:53 I changed the code to use a boolean.
908 TreeElements elements = enqueuer.resolution.getCachedElements(element); 909 TreeElements elements = enqueuer.resolution.getCachedElements(element);
909 if (elements != null) return elements; 910 if (elements != null) return elements;
910 assert(parser != null); 911 assert(parser != null);
911 Node tree = parser.parse(element); 912 Node tree = parser.parse(element);
912 validator.validate(tree); 913 validator.validate(tree);
913 elements = resolver.resolve(element); 914 elements = resolver.resolve(element);
914 if (elements != null) { 915 if (elements != null) {
915 // Only analyze nodes with a corresponding [TreeElements]. 916 // Only analyze nodes with a corresponding [TreeElements].
916 checker.check(tree, elements); 917 checker.check(tree, elements);
917 } 918 }
(...skipping 361 matching lines...) Expand 10 before | Expand all | Expand 10 after
1279 1280
1280 void close() {} 1281 void close() {}
1281 1282
1282 toString() => name; 1283 toString() => name;
1283 1284
1284 /// Convenience method for getting an [api.CompilerOutputProvider]. 1285 /// Convenience method for getting an [api.CompilerOutputProvider].
1285 static NullSink outputProvider(String name, String extension) { 1286 static NullSink outputProvider(String name, String extension) {
1286 return new NullSink('$name.$extension'); 1287 return new NullSink('$name.$extension');
1287 } 1288 }
1288 } 1289 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698