Skip to content

Commit ab49c6e

Browse files
fshcheglovCommit Queue
authored andcommitted
Deprecate getters and methods within analyzer/lib/src/dart/ast
Change-Id: I100b713897275e0a76bca5e669f73786a2a94ab5 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/437641 Commit-Queue: Konstantin Shcheglov <[email protected]> Reviewed-by: Konstantin Shcheglov <[email protected]> Reviewed-by: Brian Wilkerson <[email protected]> Reviewed-by: Paul Berry <[email protected]>
1 parent d3aa74a commit ab49c6e

File tree

177 files changed

+917
-632
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

177 files changed

+917
-632
lines changed

pkg/analysis_server/lib/src/computer/computer_call_hierarchy.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -377,7 +377,7 @@ class DartCallHierarchyComputer {
377377
node = node.propertyName;
378378
}
379379

380-
var element = ElementLocator.locate2(node);
380+
var element = ElementLocator.locate(node);
381381

382382
// Don't consider synthetic getter/setter for a field to be executable
383383
// since they don't contain any executable code.

pkg/analysis_server/lib/src/computer/computer_highlights.dart

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -805,7 +805,7 @@ class _DartUnitHighlightsComputerVisitor extends RecursiveAstVisitor<void> {
805805
exceptionParameter.name,
806806
HighlightRegionType.LOCAL_VARIABLE_DECLARATION,
807807
additionalSemanticTokenModifiers: _additionalModifiersForElement(
808-
exceptionParameter.declaredElement2,
808+
exceptionParameter.declaredElement,
809809
),
810810
);
811811
}
@@ -815,7 +815,7 @@ class _DartUnitHighlightsComputerVisitor extends RecursiveAstVisitor<void> {
815815
stackTraceParameter.name,
816816
HighlightRegionType.LOCAL_VARIABLE_DECLARATION,
817817
additionalSemanticTokenModifiers: _additionalModifiersForElement(
818-
stackTraceParameter.declaredElement2,
818+
stackTraceParameter.declaredElement,
819819
),
820820
);
821821
}
@@ -940,7 +940,7 @@ class _DartUnitHighlightsComputerVisitor extends RecursiveAstVisitor<void> {
940940
node.name,
941941
HighlightRegionType.LOCAL_VARIABLE_DECLARATION,
942942
additionalSemanticTokenModifiers: _additionalModifiersForElement(
943-
node.declaredElement2,
943+
node.declaredElement,
944944
),
945945
);
946946

@@ -1504,7 +1504,7 @@ class _DartUnitHighlightsComputerVisitor extends RecursiveAstVisitor<void> {
15041504
computer._addIdentifierRegion(
15051505
parent: node,
15061506
nameToken: node.name,
1507-
element: node.element2,
1507+
element: node.element,
15081508
);
15091509

15101510
node.typeArguments?.accept(this);
@@ -1552,7 +1552,7 @@ class _DartUnitHighlightsComputerVisitor extends RecursiveAstVisitor<void> {
15521552
if (name != null) {
15531553
// Patterns can be method tear-offs as well as getters:
15541554
// https://github.com/dart-lang/sdk/issues/59976#issuecomment-2613558317
1555-
var type = switch (node.element2) {
1555+
var type = switch (node.element) {
15561556
MethodElement() => HighlightRegionType.INSTANCE_METHOD_TEAR_OFF,
15571557
_ => HighlightRegionType.INSTANCE_GETTER_REFERENCE,
15581558
};
@@ -1688,7 +1688,7 @@ class _DartUnitHighlightsComputerVisitor extends RecursiveAstVisitor<void> {
16881688
computer._addIdentifierRegion(
16891689
parent: parent,
16901690
nameToken: node.token,
1691-
element: node.writeOrReadElement2,
1691+
element: node.writeOrReadElement,
16921692
);
16931693
}
16941694
super.visitSimpleIdentifier(node);
@@ -1849,7 +1849,7 @@ class _DartUnitHighlightsComputerVisitor extends RecursiveAstVisitor<void> {
18491849

18501850
@override
18511851
void visitVariableDeclaration(VariableDeclaration node) {
1852-
var element = node.declaredFragment?.element ?? node.declaredElement2;
1852+
var element = node.declaredFragment?.element ?? node.declaredElement;
18531853
if (element is FieldElement) {
18541854
computer._addRegion_token(
18551855
node.name,

pkg/analysis_server/lib/src/computer/computer_hover.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,13 +56,13 @@ class DartUnitHoverComputer {
5656
node is VariablePattern ||
5757
node is PatternFieldName ||
5858
node is DartPattern ||
59-
(node is LibraryDirective && node.name2 == null) ||
59+
(node is LibraryDirective && node.name == null) ||
6060
(node is SimpleIdentifier && node.parent is ImportDirective) ||
6161
node is ImportPrefixReference) {
6262
var range = _hoverRange(node, locationEntity);
6363
var hover = HoverInformation(range.offset, range.length);
6464
// element
65-
var element = ElementLocator.locate2(node);
65+
var element = ElementLocator.locate(node);
6666
if (element != null) {
6767
// short code that illustrates the element meaning.
6868
hover.elementDescription = _elementDisplayString(node, element);

pkg/analysis_server/lib/src/computer/computer_inlay_hint.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -271,7 +271,7 @@ class _DartInlayHintComputerVisitor extends GeneralizingAstVisitor<void> {
271271
return;
272272
}
273273

274-
var declaration = node.declaredElement2;
274+
var declaration = node.declaredElement;
275275
if (declaration is LocalVariableElement) {
276276
_computer._addTypePrefix(node.name, declaration.type);
277277
}
@@ -286,7 +286,7 @@ class _DartInlayHintComputerVisitor extends GeneralizingAstVisitor<void> {
286286
return;
287287
}
288288

289-
var declaration = node.declaredElement2;
289+
var declaration = node.declaredElement;
290290
if (declaration != null) {
291291
_computer._addTypePrefix(node.name, declaration.type);
292292
}

pkg/analysis_server/lib/src/computer/computer_signature.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ class DartUnitSignatureComputer {
4242
var parent = argumentList.parent;
4343
if (parent is MethodInvocation) {
4444
name = parent.methodName.name;
45-
element = ElementLocator.locate2(parent);
45+
element = ElementLocator.locate(parent);
4646
parameters =
4747
element is FunctionTypedElement ? element.formalParameters : null;
4848
} else if (parent is InstanceCreationExpression) {
@@ -51,7 +51,7 @@ class DartUnitSignatureComputer {
5151
if (constructorName != null) {
5252
name += '.${constructorName.name}';
5353
}
54-
element = ElementLocator.locate2(parent);
54+
element = ElementLocator.locate(parent);
5555
parameters =
5656
element is FunctionTypedElement ? element.formalParameters : null;
5757
} else if (parent case FunctionExpressionInvocation(

pkg/analysis_server/lib/src/computer/computer_type_arguments_signature.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,9 +44,9 @@ class DartTypeArgumentsSignatureComputer {
4444
var parent = argumentList.parent;
4545
Element? element;
4646
if (parent is NamedType) {
47-
element = parent.element2;
47+
element = parent.element;
4848
} else if (parent is MethodInvocation) {
49-
element = ElementLocator.locate2(parent.methodName);
49+
element = ElementLocator.locate(parent.methodName);
5050
}
5151
if (element is! TypeParameterizedElement ||
5252
element.typeParameters2.isEmpty) {

pkg/analysis_server/lib/src/computer/imported_elements_computer.dart

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,8 @@ class _Visitor extends UnifyingAstVisitor<void> {
7171
void visitNamedType(NamedType node) {
7272
if (node.offset <= endOffset && node.end >= startOffset) {
7373
var importPrefix = node.importPrefix;
74-
var prefix = importPrefix?.element2?.name3 ?? '';
75-
_addElement(prefix, node.element2);
74+
var prefix = importPrefix?.element?.name3 ?? '';
75+
_addElement(prefix, node.element);
7676
}
7777

7878
super.visitNamedType(node);
@@ -91,7 +91,7 @@ class _Visitor extends UnifyingAstVisitor<void> {
9191
node.offset <= endOffset &&
9292
node.end >= startOffset &&
9393
!_isConstructorDeclarationReturnType(node)) {
94-
var nodeElement = node.writeOrReadElement2;
94+
var nodeElement = node.writeOrReadElement;
9595

9696
var prefix = '';
9797
var parent = node.parent;

pkg/analysis_server/lib/src/domains/analysis/occurrences_dart.dart

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ class DartUnitOccurrencesComputerVisitor extends RecursiveAstVisitor<void> {
3939

4040
@override
4141
void visitAssignedVariablePattern(AssignedVariablePattern node) {
42-
var element = node.element2;
42+
var element = node.element;
4343
if (element != null) {
4444
_addOccurrence(element, node.name);
4545
}
@@ -100,10 +100,10 @@ class DartUnitOccurrencesComputerVisitor extends RecursiveAstVisitor<void> {
100100

101101
@override
102102
void visitDeclaredVariablePattern(DeclaredVariablePattern node) {
103-
if (node.declaredElement2 case BindPatternVariableElement(:var join2?)) {
103+
if (node.declaredElement case BindPatternVariableElement(:var join2?)) {
104104
_addOccurrence(join2.baseElement, node.name);
105105
} else {
106-
_addOccurrence(node.declaredElement2!, node.name);
106+
_addOccurrence(node.declaredElement!, node.name);
107107
}
108108

109109
super.visitDeclaredVariablePattern(node);
@@ -134,7 +134,7 @@ class DartUnitOccurrencesComputerVisitor extends RecursiveAstVisitor<void> {
134134

135135
@override
136136
void visitExtensionOverride(ExtensionOverride node) {
137-
_addOccurrence(node.element2, node.name);
137+
_addOccurrence(node.element, node.name);
138138

139139
super.visitExtensionOverride(node);
140140
}
@@ -182,7 +182,7 @@ class DartUnitOccurrencesComputerVisitor extends RecursiveAstVisitor<void> {
182182

183183
@override
184184
void visitImportPrefixReference(ImportPrefixReference node) {
185-
_addOccurrence(node.element2!, node.name);
185+
_addOccurrence(node.element!, node.name);
186186

187187
super.visitImportPrefixReference(node);
188188
}
@@ -203,7 +203,7 @@ class DartUnitOccurrencesComputerVisitor extends RecursiveAstVisitor<void> {
203203

204204
@override
205205
void visitNamedType(NamedType node) {
206-
var element = node.element2;
206+
var element = node.element;
207207
if (element != null) {
208208
_addOccurrence(element, node.name);
209209
}
@@ -213,7 +213,7 @@ class DartUnitOccurrencesComputerVisitor extends RecursiveAstVisitor<void> {
213213

214214
@override
215215
void visitPatternField(PatternField node) {
216-
var element = node.element2;
216+
var element = node.element;
217217
var pattern = node.pattern;
218218
// If no explicit field name, use the variables name.
219219
var name =
@@ -257,7 +257,7 @@ class DartUnitOccurrencesComputerVisitor extends RecursiveAstVisitor<void> {
257257
return;
258258
}
259259

260-
var element = node.writeOrReadElement2;
260+
var element = node.writeOrReadElement;
261261
if (element != null) {
262262
_addOccurrence(element, node.token);
263263
}

pkg/analysis_server/lib/src/lsp/handlers/custom/handler_imports.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ class ImportsHandler
6363
return success(null);
6464
}
6565

66-
var element = ElementLocator.locate2(node);
66+
var element = ElementLocator.locate(node);
6767
if (element == null) {
6868
return success(null);
6969
}

pkg/analysis_server/lib/src/lsp/handlers/custom/handler_super.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ class SuperHandler
5050
return success(null);
5151
}
5252

53-
var element = ElementLocator.locate2(node);
53+
var element = ElementLocator.locate(node);
5454
if (element == null) {
5555
return success(null);
5656
}

0 commit comments

Comments
 (0)