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

Side by Side Diff: pkg/front_end/lib/src/fasta/dill/dill_class_builder.dart

Issue 2691523002: Ensure locations are always provided, but don't store them yet. (Closed)
Patch Set: Missing file. Created 3 years, 10 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) 2016, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2016, 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 library fasta.dill_class_builder; 5 library fasta.dill_class_builder;
6 6
7 import 'package:kernel/ast.dart' show 7 import 'package:kernel/ast.dart' show
8 Class, 8 Class,
9 Constructor, 9 Constructor,
10 Member, 10 Member,
(...skipping 11 matching lines...) Expand all
22 DillLibraryBuilder; 22 DillLibraryBuilder;
23 23
24 class DillClassBuilder extends KernelClassBuilder { 24 class DillClassBuilder extends KernelClassBuilder {
25 final Class cls; 25 final Class cls;
26 26
27 final Map<String, Builder> constructors = <String, Builder>{}; 27 final Map<String, Builder> constructors = <String, Builder>{};
28 28
29 DillClassBuilder(Class cls, DillLibraryBuilder parent) 29 DillClassBuilder(Class cls, DillLibraryBuilder parent)
30 : cls = cls, 30 : cls = cls,
31 super(null, null, cls.name, null, null, null, <String, Builder>{}, 31 super(null, null, cls.name, null, null, null, <String, Builder>{},
32 null, parent); 32 null, parent, cls.fileOffset);
33 33
34 void addMember(Member member) { 34 void addMember(Member member) {
35 DillMemberBuilder builder = new DillMemberBuilder(member, this); 35 DillMemberBuilder builder = new DillMemberBuilder(member, this);
36 String name = member.name.name; 36 String name = member.name.name;
37 if (member is Constructor || 37 if (member is Constructor ||
38 (member is Procedure && member.kind == ProcedureKind.Factory)) { 38 (member is Procedure && member.kind == ProcedureKind.Factory)) {
39 constructors[name] = builder; 39 constructors[name] = builder;
40 } else { 40 } else {
41 DillMemberBuilder existing = members[name]; 41 DillMemberBuilder existing = members[name];
42 if (existing == null) { 42 if (existing == null) {
43 members[name] = builder; 43 members[name] = builder;
44 } else { 44 } else {
45 existing.next = builder; 45 existing.next = builder;
46 } 46 }
47 } 47 }
48 } 48 }
49 49
50 Builder findConstructorOrFactory(String name) => constructors[name]; 50 Builder findConstructorOrFactory(String name) => constructors[name];
51 } 51 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698