Skip to content

Commit 09b78d2

Browse files
stereotype441Commit Queue
authored andcommitted
[analyzer] Fix documentation for GeneralizingElementVisitor2.
The tree describing the type structure assumed by the visitor was out of date: - It included a reference to `PromotableElement`, which no longer exists. - It was missing references to `EnumElement`, `ExtensionElement`, `ExtensionTypeElement`, `GenericFunctionTypeElement`, `LocalVariableElement`, `MixinElement`, `PropertyAccessorElement`, and `TopLevelFunctionElement`. - It incorrectly listed `FormalParameterElement` under `VariableElement`. In fact, `visitFormalParameterElement` redirects to `visitElement`. This is a step towards enabling the `comment_references` lint in the analyzer codebase. Change-Id: I6a6a69646cefded105739f207df1554d4beb717d Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/450741 Auto-Submit: Paul Berry <[email protected]> Commit-Queue: Konstantin Shcheglov <[email protected]> Reviewed-by: Brian Wilkerson <[email protected]> Reviewed-by: Konstantin Shcheglov <[email protected]>
1 parent 026362d commit 09b78d2

File tree

1 file changed

+17
-11
lines changed

1 file changed

+17
-11
lines changed

pkg/analyzer/lib/dart/element/visitor2.dart

Lines changed: 17 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -47,27 +47,33 @@ import 'package:analyzer/dart/element/element.dart';
4747
/// <pre>
4848
/// [Element]
4949
/// [ClassElement]
50+
/// [EnumElement]
5051
/// [ExecutableElement]
51-
/// [ConstructorElement]
52+
/// [ConstructorElement]
53+
/// [LocalFunctionElement]
54+
/// [MethodElement]
55+
/// [PropertyAccessorElement]
5256
/// [GetterElement]
53-
/// [LocalFunctionElement]
54-
/// [MethodElement]
5557
/// [SetterElement]
58+
/// [TopLevelFunctionElement]
59+
/// [ExtensionElement]
60+
/// [ExtensionTypeElement]
61+
/// [FormalParameterElement]
62+
/// [FieldFormalParameterElement]
63+
/// [SuperFormalParameterElement]
64+
/// [GenericFunctionTypeElement]
5665
/// [LabelElement]
5766
/// [LibraryElement]
67+
/// [MixinElement]
5868
/// [MultiplyDefinedElement]
5969
/// [PrefixElement]
6070
/// [TypeAliasElement]
6171
/// [TypeParameterElement]
6272
/// [VariableElement]
63-
/// [PropertyInducingElement]
64-
/// [FieldElement]
65-
/// [TopLevelVariableElement]
66-
/// [PromotableElement]
67-
/// [LocalVariableElement]
68-
/// [FormalParameterElement]
69-
/// [FieldFormalParameterElement]
70-
/// [SuperFormalParameterElement]
73+
/// [LocalVariableElement]
74+
/// [PropertyInducingElement]
75+
/// [FieldElement]
76+
/// [TopLevelVariableElement]
7177
/// </pre>
7278
///
7379
/// Subclasses that override a visit method must either invoke the overridden

0 commit comments

Comments
 (0)