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

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

Issue 11413219: Canonicalize raw type (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Updated cf. comment Created 8 years 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 dart_backend; 5 part of dart_backend;
6 6
7 // TODO(ahe): This class is simply wrong. This backend should use 7 // TODO(ahe): This class is simply wrong. This backend should use
8 // elements when it can, not AST nodes. Perhaps a [Map<Element, 8 // elements when it can, not AST nodes. Perhaps a [Map<Element,
9 // TreeElements>] is what is needed. 9 // TreeElements>] is what is needed.
10 class ElementAst { 10 class ElementAst {
(...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after
130 * 2) The code has typedefs in right hand side of IS checks, 130 * 2) The code has typedefs in right hand side of IS checks,
131 * 3) The code has classes which extend typedefs, have type arguments typedefs 131 * 3) The code has classes which extend typedefs, have type arguments typedefs
132 * or type variable bounds typedefs. 132 * or type variable bounds typedefs.
133 * These restrictions can be less strict. 133 * These restrictions can be less strict.
134 */ 134 */
135 bool isSafeToRemoveTypeDeclarations( 135 bool isSafeToRemoveTypeDeclarations(
136 Map<ClassElement, Set<Element>> classMembers) { 136 Map<ClassElement, Set<Element>> classMembers) {
137 Set<DartType> processedTypes = new Set<DartType>(); 137 Set<DartType> processedTypes = new Set<DartType>();
138 List<DartType> workQueue = new List<DartType>(); 138 List<DartType> workQueue = new List<DartType>();
139 workQueue.addAll( 139 workQueue.addAll(
140 classMembers.keys.map((classElement) => classElement.type)); 140 classMembers.keys.map((classElement) => classElement.thisType));
141 workQueue.addAll(compiler.resolverWorld.isChecks); 141 workQueue.addAll(compiler.resolverWorld.isChecks);
142 Element typeErrorElement = 142 Element typeErrorElement =
143 compiler.coreLibrary.find(new SourceString('TypeError')); 143 compiler.coreLibrary.find(new SourceString('TypeError'));
144 DartType typeErrorType = typeErrorElement.computeType(compiler); 144 DartType typeErrorType = typeErrorElement.computeType(compiler);
145 if (workQueue.indexOf(typeErrorType) != -1) { 145 if (workQueue.indexOf(typeErrorType) != -1) {
146 return false; 146 return false;
147 } 147 }
148 148
149 void processTypeArguments(Element classElement, NodeList typeArguments) { 149 void processTypeArguments(Element classElement, NodeList typeArguments) {
150 if (typeArguments == null) return; 150 if (typeArguments == null) return;
(...skipping 402 matching lines...) Expand 10 before | Expand all | Expand 10 after
553 } 553 }
554 554
555 compareElements(e0, e1) { 555 compareElements(e0, e1) {
556 int result = compareBy((e) => e.getLibrary().uri.toString())(e0, e1); 556 int result = compareBy((e) => e.getLibrary().uri.toString())(e0, e1);
557 if (result != 0) return result; 557 if (result != 0) return result;
558 return compareBy((e) => e.position().charOffset)(e0, e1); 558 return compareBy((e) => e.position().charOffset)(e0, e1);
559 } 559 }
560 560
561 List<Element> sortElements(Collection<Element> elements) => 561 List<Element> sortElements(Collection<Element> elements) =>
562 sorted(elements, compareElements); 562 sorted(elements, compareElements);
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698