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

Side by Side Diff: pkg/kernel/lib/ast.dart

Issue 2781893004: Remove definitions from Kernel canonical names. (Closed)
Patch Set: Update test expectations. Created 3 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
« no previous file with comments | « no previous file | pkg/kernel/lib/canonical_name.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 /// ----------------------------------------------------------------------- 5 /// -----------------------------------------------------------------------
6 /// ERROR HANDLING 6 /// ERROR HANDLING
7 /// ----------------------------------------------------------------------- 7 /// -----------------------------------------------------------------------
8 /// 8 ///
9 /// As a rule of thumb, errors that can be detected statically are handled by 9 /// As a rule of thumb, errors that can be detected statically are handled by
10 /// the frontend, typically by translating the erroneous code into a 'throw' or 10 /// the frontend, typically by translating the erroneous code into a 'throw' or
(...skipping 979 matching lines...) Expand 10 before | Expand all | Expand 10 after
990 void set isConst(bool value) { 990 void set isConst(bool value) {
991 flags = value ? (flags | FlagConst) : (flags & ~FlagConst); 991 flags = value ? (flags | FlagConst) : (flags & ~FlagConst);
992 } 992 }
993 993
994 bool get isInstanceMember => !isStatic; 994 bool get isInstanceMember => !isStatic;
995 bool get isGetter => kind == ProcedureKind.Getter; 995 bool get isGetter => kind == ProcedureKind.Getter;
996 bool get isSetter => kind == ProcedureKind.Setter; 996 bool get isSetter => kind == ProcedureKind.Setter;
997 bool get isAccessor => isGetter || isSetter; 997 bool get isAccessor => isGetter || isSetter;
998 bool get hasGetter => kind != ProcedureKind.Setter; 998 bool get hasGetter => kind != ProcedureKind.Setter;
999 bool get hasSetter => kind == ProcedureKind.Setter; 999 bool get hasSetter => kind == ProcedureKind.Setter;
1000 bool get isFactory => kind == ProcedureKind.Factory;
1000 1001
1001 accept(MemberVisitor v) => v.visitProcedure(this); 1002 accept(MemberVisitor v) => v.visitProcedure(this);
1002 1003
1003 acceptReference(MemberReferenceVisitor v) => v.visitProcedureReference(this); 1004 acceptReference(MemberReferenceVisitor v) => v.visitProcedureReference(this);
1004 1005
1005 visitChildren(Visitor v) { 1006 visitChildren(Visitor v) {
1006 visitList(annotations, v); 1007 visitList(annotations, v);
1007 name?.accept(v); 1008 name?.accept(v);
1008 function?.accept(v); 1009 function?.accept(v);
1009 } 1010 }
(...skipping 3221 matching lines...) Expand 10 before | Expand all | Expand 10 after
4231 /// library has not been assigned a canonical name yet. 4232 /// library has not been assigned a canonical name yet.
4232 /// 4233 ///
4233 /// Returns `null` if the library is `null`. 4234 /// Returns `null` if the library is `null`.
4234 CanonicalName getCanonicalNameOfLibrary(Library library) { 4235 CanonicalName getCanonicalNameOfLibrary(Library library) {
4235 if (library == null) return null; 4236 if (library == null) return null;
4236 if (library.canonicalName == null) { 4237 if (library.canonicalName == null) {
4237 throw '$library has no canonical name'; 4238 throw '$library has no canonical name';
4238 } 4239 }
4239 return library.canonicalName; 4240 return library.canonicalName;
4240 } 4241 }
OLDNEW
« no previous file with comments | « no previous file | pkg/kernel/lib/canonical_name.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698