Chromium Code Reviews| OLD | NEW |
|---|---|
| 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 Loading... | |
| 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 Loading... | |
| 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 } |
| OLD | NEW |