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

Side by Side Diff: pkg/compiler/lib/src/serialization/modelz.dart

Issue 2005293003: Compute import paths using ImportElement and ExportElement (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Compute members eagerly. Created 4 years, 7 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
OLDNEW
1 // Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2015, 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 /// Implementation of the element model used for deserialiation. 5 /// Implementation of the element model used for deserialiation.
6 /// 6 ///
7 /// These classes are created by [ElementDeserializer] triggered by the 7 /// These classes are created by [ElementDeserializer] triggered by the
8 /// [Deserializer]. 8 /// [Deserializer].
9 9
10 library dart2js.serialization.modelz; 10 library dart2js.serialization.modelz;
(...skipping 828 matching lines...) Expand 10 before | Expand all | Expand 10 after
839 return null; 839 return null;
840 } 840 }
841 841
842 @override 842 @override
843 ClassElement get superclass => supertype != null ? supertype.element : null; 843 ClassElement get superclass => supertype != null ? supertype.element : null;
844 844
845 @override 845 @override
846 void ensureResolved(Resolution resolution) { 846 void ensureResolved(Resolution resolution) {
847 if (!_isResolved) { 847 if (!_isResolved) {
848 _isResolved = true; 848 _isResolved = true;
849 class_members.MembersCreator 849 // TODO(johnniwinther): Avoid eager computation of all members. `call` is
850 .computeClassMembersByName(resolution, this, Identifiers.call); 850 // always needed, but the remaining should be computed on-demand or on
851 // type instantiation.
Siggi Cherem (dart-lang) 2016/05/26 22:38:08 nit: fix indent
Johnni Winther 2016/05/27 08:52:25 Done.
852 class_members.MembersCreator.computeAllClassMembers(resolution, this);
851 resolution.registerClass(this); 853 resolution.registerClass(this);
852 } 854 }
853 } 855 }
854 } 856 }
855 857
856 class ClassElementZ extends DeserializedElementZ 858 class ClassElementZ extends DeserializedElementZ
857 with 859 with
858 AnalyzableElementMixin, 860 AnalyzableElementMixin,
859 AstElementMixinZ, 861 AstElementMixinZ,
860 ClassElementCommon, 862 ClassElementCommon,
(...skipping 1378 matching lines...) Expand 10 before | Expand all | Expand 10 after
2239 } 2241 }
2240 2242
2241 @override 2243 @override
2242 ElementKind get kind => ElementKind.PREFIX; 2244 ElementKind get kind => ElementKind.PREFIX;
2243 2245
2244 @override 2246 @override
2245 Element lookupLocalMember(String memberName) { 2247 Element lookupLocalMember(String memberName) {
2246 return _unsupported('lookupLocalMember'); 2248 return _unsupported('lookupLocalMember');
2247 } 2249 }
2248 } 2250 }
OLDNEW
« no previous file with comments | « pkg/compiler/lib/src/library_loader.dart ('k') | tests/compiler/dart2js/serialization/test_data.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698