Skip to content

Commit 7218145

Browse files
fshcheglovCommit Queue
authored andcommitted
Deprecate appendToWithoutDelimiters2, correspondingSetter2, unnamedConstructor2, getNamedConstructor2, entryPoint2, exportedLibraries2, getClass2, getEnum2, getMixin2, exportedLibrary2, accessibleExtensions2, importedLibraries2, importedLibrary2, isImplementableIn2, join2, variable3, getter2, setter2, correspondingGetter2, superConstructorParameter2, aliasedElement2, typeParameters2, constantInitializer2, and isNonSubtypableClass2
Change-Id: I091804b9f200f525baf3979773dd489e3be17697 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/439640 Commit-Queue: Konstantin Shcheglov <scheglov@google.com> Reviewed-by: Konstantin Shcheglov <scheglov@google.com> Reviewed-by: Samuel Rawlins <srawlins@google.com>
1 parent 3e7e667 commit 7218145

File tree

156 files changed

+970
-654
lines changed

Some content is hidden

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

156 files changed

+970
-654
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,7 @@ class CiderCompletionComputer {
173173
var importedLibraries =
174174
target.withEnclosing2
175175
.expand((fragment) => fragment.libraryImports2)
176-
.map((import) => import.importedLibrary2)
176+
.map((import) => import.importedLibrary)
177177
.nonNulls
178178
.toSet();
179179
for (var importedLibrary in importedLibraries) {

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ class LibraryElementSuggestionBuilder
9999

100100
@override
101101
void visitGetterElement(GetterElement element) {
102-
var variable = element.variable3;
102+
var variable = element.variable;
103103
if (opType.includeReturnValueSuggestions ||
104104
(opType.includeAnnotationSuggestions &&
105105
variable != null &&
@@ -130,15 +130,15 @@ class LibraryElementSuggestionBuilder
130130
visitMixinElement(MixinElement element) {
131131
AstNode node = request.target.containingNode;
132132
if (node is ImplementsClause &&
133-
!element.isImplementableIn2(request.libraryElement)) {
133+
!element.isImplementableIn(request.libraryElement)) {
134134
return;
135135
}
136136
_visitInterfaceElement(element);
137137
}
138138

139139
@override
140140
void visitSetterElement(SetterElement element) {
141-
var variable = element.variable3;
141+
var variable = element.variable;
142142
if (opType.includeReturnValueSuggestions ||
143143
(opType.includeAnnotationSuggestions &&
144144
variable != null &&
@@ -228,7 +228,7 @@ class LibraryElementSuggestionBuilder
228228
field.isAccessibleIn(request.libraryElement) &&
229229
typeSystem.isSubtypeOf(field.type, contextType)) {
230230
if (field.isSynthetic) {
231-
var getter = field.getter2;
231+
var getter = field.getter;
232232
if (getter != null) {
233233
builder.suggestGetter(
234234
getter,

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

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -99,8 +99,8 @@ class CanRenameResponse {
9999
var oldStateName = '${element.displayName}State';
100100
var library = element.library;
101101
var state =
102-
library.getClass2(oldStateName) ??
103-
library.getClass2('_$oldStateName');
102+
library.getClass(oldStateName) ??
103+
library.getClass('_$oldStateName');
104104
if (state != null) {
105105
var flutterWidgetStateNewName = '${newName}State';
106106
// If the State was private, ensure that it stays private.
@@ -131,8 +131,8 @@ class CheckNameResponse {
131131
var element = canRename.refactoringElement.element;
132132
if (element is PropertyInducingElement && element.isSynthetic) {
133133
var property = element;
134-
var getter = property.getter2;
135-
var setter = property.setter2;
134+
var getter = property.getter;
135+
var setter = property.setter;
136136
elements.addIfNotNull(getter);
137137
elements.addIfNotNull(setter);
138138
} else {
@@ -235,7 +235,7 @@ class CheckNameResponse {
235235
) async {
236236
var infos = <ReplaceInfo>[];
237237
if (element is PropertyInducingElement && element.isSynthetic) {
238-
var getter = element.getter2;
238+
var getter = element.getter;
239239
if (getter != null) {
240240
infos.add(
241241
ReplaceInfo(
@@ -245,7 +245,7 @@ class CheckNameResponse {
245245
),
246246
);
247247
}
248-
var setter = element.setter2;
248+
var setter = element.setter;
249249
if (setter != null) {
250250
infos.add(
251251
ReplaceInfo(
@@ -436,7 +436,7 @@ class CiderRenameComputer {
436436
return null;
437437
}
438438
if (element is PropertyAccessorElement) {
439-
element = element.variable3;
439+
element = element.variable;
440440
if (element == null) {
441441
return null;
442442
}

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

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -33,16 +33,15 @@ class DartDocumentationComputer {
3333
element,
3434
...overridden.superElements,
3535
...overridden.interfaceElements,
36-
if (element case PropertyAccessorElement(variable3: var variable?))
37-
variable,
36+
if (element case PropertyAccessorElement(:var variable?)) variable,
3837
];
3938
for (var candidate in candidates) {
4039
if (candidate.documentationCommentOrNull != null) {
4140
documentedElement = candidate;
4241
break;
4342
}
4443
if (documentedGetter == null && candidate is SetterElement) {
45-
var getter = candidate.correspondingGetter2;
44+
var getter = candidate.correspondingGetter;
4645
if (getter != null && getter.documentationComment != null) {
4746
documentedGetter = getter;
4847
}
@@ -108,9 +107,9 @@ extension on Element {
108107
// Treat a super formal parameter like a field formal parameter if it's
109108
// eventually assigned to a field, but as any other formal parameter if it
110109
// isn't.
111-
var superParameter = self.superConstructorParameter2;
110+
var superParameter = self.superConstructorParameter;
112111
while (superParameter is SuperFormalParameterElement) {
113-
superParameter = superParameter.superConstructorParameter2;
112+
superParameter = superParameter.superConstructorParameter;
114113
}
115114
if (superParameter is FieldFormalParameterElement) {
116115
return superParameter.field;

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -271,7 +271,7 @@ class DartUnitHighlightsComputer {
271271
type = HighlightRegionType.TOP_LEVEL_VARIABLE_DECLARATION;
272272
} else if (element is GetterElement) {
273273
var accessor = element;
274-
var variable = accessor.variable3;
274+
var variable = accessor.variable;
275275
if (variable is TopLevelVariableElement) {
276276
type = HighlightRegionType.TOP_LEVEL_GETTER_REFERENCE;
277277
} else if (variable is FieldElement && variable.isEnumConstant) {
@@ -283,7 +283,7 @@ class DartUnitHighlightsComputer {
283283
}
284284
} else if (element is SetterElement) {
285285
var accessor = element;
286-
var variable = accessor.variable3;
286+
var variable = accessor.variable;
287287
if (variable is TopLevelVariableElement) {
288288
type = HighlightRegionType.TOP_LEVEL_SETTER_REFERENCE;
289289
} else if (variable is FieldElement && variable.isEnumConstant) {

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -381,7 +381,7 @@ class ImportElementsComputer {
381381
if (libraryImport == null) {
382382
return false;
383383
}
384-
var library = libraryImport.importedLibrary2;
384+
var library = libraryImport.importedLibrary;
385385
return library != null &&
386386
library.firstFragment.source.fullName == importedElements.path &&
387387
(import.prefix?.name ?? '') == importedElements.prefix;

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

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

101101
@override
102102
void visitDeclaredVariablePattern(DeclaredVariablePattern node) {
103-
if (node.declaredElement case BindPatternVariableElement(:var join2?)) {
104-
_addOccurrence(join2.baseElement, node.name);
103+
if (node.declaredElement case BindPatternVariableElement(:var join?)) {
104+
_addOccurrence(join.baseElement, node.name);
105105
} else {
106106
_addOccurrence(node.declaredElement!, node.name);
107107
}
@@ -307,9 +307,9 @@ class DartUnitOccurrencesComputerVisitor extends RecursiveAstVisitor<void> {
307307
if (canonicalElement is FieldFormalParameterElement) {
308308
canonicalElement = canonicalElement.field;
309309
} else if (canonicalElement case PropertyAccessorElement(
310-
:var variable3?,
311-
) when !variable3.isSynthetic) {
312-
canonicalElement = variable3;
310+
:var variable?,
311+
) when !variable.isSynthetic) {
312+
canonicalElement = variable;
313313
}
314314
return canonicalElement?.baseElement;
315315
}

pkg/analysis_server/lib/src/handler/legacy/search_find_element_references.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ class SearchFindElementReferencesHandler extends LegacyHandler {
3636
element = element.field;
3737
}
3838
if (element is PropertyAccessorElement) {
39-
element = element.variable3;
39+
element = element.variable;
4040
}
4141
// respond
4242
var searchId = (server.nextSearchId++).toString();

pkg/analysis_server/lib/src/lsp/handlers/handler_document_color_presentation.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -249,7 +249,7 @@ class DocumentColorPresentationHandler
249249
parent is PrefixedIdentifier ? parent.element : node.element;
250250

251251
return switch (element) {
252-
PropertyAccessorElement(:var variable3) => variable3?.isConst ?? false,
252+
PropertyAccessorElement(:var variable) => variable?.isConst ?? false,
253253
VariableElement() => element.isConst,
254254
_ => false,
255255
};

pkg/analysis_server/lib/src/lsp/handlers/handler_references.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ class ReferencesHandler
6969

7070
var element = switch (node?.getElement()) {
7171
FieldFormalParameterElement(:var field?) => field,
72-
PropertyAccessorElement(:var variable3?) => variable3,
72+
PropertyAccessorElement(:var variable?) => variable,
7373
var element => element,
7474
};
7575

0 commit comments

Comments
 (0)