Skip to content

Commit 05da3e1

Browse files
scheglovCommit Queue
authored andcommitted
Elements. Deprecate Element.enclosingElement2, use enclosingElement.
Change-Id: I055b1f9dd1c00605d813effcbbadb9cea874129a Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/428860 Reviewed-by: Brian Wilkerson <[email protected]> Commit-Queue: Konstantin Shcheglov <[email protected]>
1 parent c6722fc commit 05da3e1

File tree

154 files changed

+575
-442
lines changed

Some content is hidden

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

154 files changed

+575
-442
lines changed

pkg/analysis_server/lib/src/cider/local_library_contributor.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ class LibraryElementSuggestionBuilder
104104
(opType.includeAnnotationSuggestions &&
105105
variable != null &&
106106
variable.isConst)) {
107-
var parent = element.enclosingElement2;
107+
var parent = element.enclosingElement;
108108
if (parent is InterfaceElement || parent is ExtensionElement) {
109109
if (element.isSynthetic) {
110110
if (variable is FieldElement) {
@@ -143,7 +143,7 @@ class LibraryElementSuggestionBuilder
143143
(opType.includeAnnotationSuggestions &&
144144
variable != null &&
145145
variable.isConst)) {
146-
var parent = element.enclosingElement2;
146+
var parent = element.enclosingElement;
147147
if (parent is InterfaceElement || parent is ExtensionElement) {
148148
if (!element.isSynthetic) {
149149
builder.suggestSetter(element, inheritanceDistance: 0.0);

pkg/analysis_server/lib/src/cider/rename.dart

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ class CanRenameResponse {
7474
RefactoringStatus result,
7575
String newName,
7676
) {
77-
var parentClass = element.enclosingElement2;
77+
var parentClass = element.enclosingElement;
7878
// Check if the "newName" is the name of the enclosing class.
7979
if (parentClass.name3 == newName) {
8080
result.addError(
@@ -362,7 +362,7 @@ class CheckNameResponse {
362362

363363
Future<CiderReplaceMatch?> _replaceSyntheticConstructor() async {
364364
var element = canRename.refactoringElement.element;
365-
var interfaceElement = element.enclosingElement2!;
365+
var interfaceElement = element.enclosingElement!;
366366

367367
var fileResolver = canRename._fileResolver;
368368
var libraryPath = interfaceElement.library2!.firstFragment.source.fullName;
@@ -452,7 +452,7 @@ class CiderRenameComputer {
452452
}
453453

454454
bool _canRenameElement(Element element) {
455-
var enclosingElement = element.enclosingElement2;
455+
var enclosingElement = element.enclosingElement;
456456
if (element is ConstructorElement) {
457457
return true;
458458
}

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ class CallHierarchyItem {
108108
file = element.firstFragment.libraryFragment!.source.fullName,
109109
kind = CallHierarchyKind.forElement(element) {
110110
var enclosingElement =
111-
element.enclosingElement2 ??
111+
element.enclosingElement ??
112112
element.firstFragment.enclosingFragment?.element;
113113
var container =
114114
enclosingElement != null ? _getContainer(enclosingElement) : null;
@@ -145,7 +145,7 @@ class CallHierarchyItem {
145145
static Element _nonSynthetic(Element element) {
146146
element = element.nonSynthetic2;
147147
if (element.isSynthetic) {
148-
element = element.enclosingElement2 ?? element;
148+
element = element.enclosingElement ?? element;
149149
}
150150
return element;
151151
}

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ class ColorComputer {
7474

7575
var constructor = expression.constructorName;
7676
var staticElement = constructor.element;
77-
var classElement = staticElement?.enclosingElement2;
77+
var classElement = staticElement?.enclosingElement;
7878
var className = classElement?.name3;
7979
var constructorName = constructor.name?.name;
8080
var constructorArgs = expression.argumentList.arguments.toList();

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ class DartDocumentationComputer {
1919
}) {
2020
var element = switch (elementBeingDocumented) {
2121
FieldFormalParameterElement() => elementBeingDocumented.field2,
22-
FormalParameterElement() => elementBeingDocumented.enclosingElement2,
22+
FormalParameterElement() => elementBeingDocumented.enclosingElement,
2323
_ => elementBeingDocumented,
2424
};
2525
if (element == null) {
@@ -68,9 +68,9 @@ class DartDocumentationComputer {
6868
includeSummary: includeSummary,
6969
);
7070

71-
var documentedElementClass = documentedElement.enclosingElement2;
71+
var documentedElementClass = documentedElement.enclosingElement;
7272
if (documentedElementClass != null &&
73-
documentedElementClass != element.enclosingElement2) {
73+
documentedElementClass != element.enclosingElement) {
7474
var documentedClass = documentedElementClass.displayName;
7575
result.full = '${result.full}\n\nCopied from `$documentedClass`.';
7676
}

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ class DartUnitHoverComputer {
7171
hover.isDeprecated = a.metadata2.hasDeprecated;
7272
}
7373
// not local element
74-
if (element.enclosingElement2 is! ExecutableElement) {
74+
if (element.enclosingElement is! ExecutableElement) {
7575
// containing class
7676
hover.containingClassDescription = _containingClass(element);
7777
// containing library
@@ -211,7 +211,7 @@ class DartUnitHoverComputer {
211211
return null;
212212
}
213213
var parameter = node.correspondingParameter;
214-
return switch (parameter?.enclosingElement2) {
214+
return switch (parameter?.enclosingElement) {
215215
// Expressions passed as arguments to setters and binary expressions
216216
// will have parameters here but we don't want them to show as such in
217217
// hovers because information about those functions are already available

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ import 'package:collection/collection.dart';
1111

1212
/// Return the elements that the given [element] overrides.
1313
OverriddenElements findOverriddenElements(Element element) {
14-
if (element.enclosingElement2 is InterfaceElement) {
14+
if (element.enclosingElement is InterfaceElement) {
1515
return _OverriddenElementsFinder(element).find();
1616
}
1717
return OverriddenElements(element, <Element>[], <Element>[]);
@@ -123,7 +123,7 @@ class _OverriddenElementsFinder {
123123
final Set<InterfaceElement> _visited = {};
124124

125125
factory _OverriddenElementsFinder(Element seed) {
126-
var class_ = seed.enclosingElement2 as InterfaceElement;
126+
var class_ = seed.enclosingElement as InterfaceElement;
127127
var library = class_.library2;
128128
var name = seed.displayName;
129129
List<ElementKind> kinds;

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ class _Visitor extends UnifyingAstVisitor<void> {
115115
if (element is PrefixElement) {
116116
return;
117117
}
118-
if (element.enclosingElement2 is! LibraryElement) {
118+
if (element.enclosingElement is! LibraryElement) {
119119
return;
120120
}
121121

pkg/analysis_server/lib/src/lsp/handlers/custom/editable_arguments/editable_arguments_mixin.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ mixin EditableArgumentsMixin {
164164

165165
/// Returns the name of an enum constant prefixed with the enum name.
166166
static String? getQualifiedEnumConstantName(FieldElement enumConstant) {
167-
var enumName = enumConstant.enclosingElement2.name3;
167+
var enumName = enumConstant.enclosingElement.name3;
168168
var name = enumConstant.name3;
169169
return enumName != null && name != null ? '$enumName.$name' : null;
170170
}
@@ -182,7 +182,7 @@ extension on InvocationExpressionImpl {
182182
// We only support @widgetFactory on extension methods.
183183
var element = switch (function) {
184184
Identifier(:var element)
185-
when element?.enclosingElement2 is ExtensionElement =>
185+
when element?.enclosingElement is ExtensionElement =>
186186
element,
187187
_ => null,
188188
};

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
@@ -82,7 +82,7 @@ class ImportsHandler
8282
}
8383
}
8484

85-
var enclosingElement = element.enclosingElement2;
85+
var enclosingElement = element.enclosingElement;
8686
if (enclosingElement is ExtensionElement) {
8787
element = enclosingElement;
8888
}

0 commit comments

Comments
 (0)