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

Unified Diff: pkg/analyzer/lib/src/summary/summarize_elements.dart

Issue 1944023002: Include all static class members in UnlinkedPublicName.members. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Fix incorrect upload (previously included changes from CL 1944773002) Created 4 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 side-by-side diff with in-line comments
Download patch
Index: pkg/analyzer/lib/src/summary/summarize_elements.dart
diff --git a/pkg/analyzer/lib/src/summary/summarize_elements.dart b/pkg/analyzer/lib/src/summary/summarize_elements.dart
index b8828ecb3051aec04325fb32c5ea22e934149293..f3aef149697be333b805a9b53d4c28cca29fd78b 100644
--- a/pkg/analyzer/lib/src/summary/summarize_elements.dart
+++ b/pkg/analyzer/lib/src/summary/summarize_elements.dart
@@ -355,7 +355,7 @@ class _CompilationUnitSerializer {
kind: ReferenceKind.classOrEnum,
name: cls.name,
numTypeParameters: cls.typeParameters.length,
- members: serializeClassConstMembers(cls)));
+ members: serializeClassStaticMembers(cls)));
}
}
for (ClassElement enm in compilationUnit.enums) {
@@ -363,7 +363,7 @@ class _CompilationUnitSerializer {
names.add(new UnlinkedPublicNameBuilder(
kind: ReferenceKind.classOrEnum,
name: enm.name,
- members: serializeClassConstMembers(enm)));
+ members: serializeClassStaticMembers(enm)));
}
}
for (FunctionElement function in compilationUnit.functions) {
@@ -594,11 +594,11 @@ class _CompilationUnitSerializer {
}
/**
- * If [cls] is a class, return the list of its members available for
- * constants - static constant fields, static methods and constructors.
- * Otherwise return `null`.
+ * If [cls] is a class, return the list of its static members - static
+ * constant fields, static methods and constructors. Otherwise return `null`.
*/
- List<UnlinkedPublicNameBuilder> serializeClassConstMembers(ClassElement cls) {
+ List<UnlinkedPublicNameBuilder> serializeClassStaticMembers(
+ ClassElement cls) {
if (cls.isMixinApplication) {
// Mixin application members can't be determined directly from the AST so
// we can't store them in UnlinkedPublicName.
@@ -607,16 +607,6 @@ class _CompilationUnitSerializer {
}
if (cls.kind == ElementKind.CLASS) {
List<UnlinkedPublicNameBuilder> bs = <UnlinkedPublicNameBuilder>[];
- for (FieldElement field in cls.fields) {
- if (field.isStatic && field.isConst && field.isPublic) {
- // TODO(paulberry): include non-consts
- // TODO(paulberry): should numTypeParameters include class params?
- bs.add(new UnlinkedPublicNameBuilder(
- name: field.name,
- kind: ReferenceKind.propertyAccessor,
- numTypeParameters: 0));
- }
- }
for (MethodElement method in cls.methods) {
if (method.isStatic && method.isPublic) {
// TODO(paulberry): should numTypeParameters include class params?
@@ -629,9 +619,7 @@ class _CompilationUnitSerializer {
for (PropertyAccessorElement accessor in cls.accessors) {
if (accessor.isStatic &&
accessor.isGetter &&
- accessor.isPublic &&
- !accessor.isSynthetic) {
- // TODO(paulberry): combine with field code above.
+ accessor.isPublic) {
// TODO(paulberry): should numTypeParameters include class params?
bs.add(new UnlinkedPublicNameBuilder(
name: accessor.name, kind: ReferenceKind.propertyAccessor));
« no previous file with comments | « pkg/analyzer/lib/src/summary/public_namespace_computer.dart ('k') | pkg/analyzer/test/src/summary/summary_common.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698