Skip to content

Commit 238cd4c

Browse files
fshcheglovCommit Queue
authored andcommitted
Deprecate constants2, lookUpInheritedMethod2, lookUpInheritedMethod2, classes2, classes2, extensionTypes2, extensions2, functions2, libraryExports2, libraryImports2, libraryImports2, mixins2, topLevelVariables2, typeAliases2, join2, isNonSubtypableClass2, and typeParameters2.
Change-Id: Ia4b5504d91548b561e2e2434e711c40b2dc5e495 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/440100 Reviewed-by: Konstantin Shcheglov <[email protected]> Commit-Queue: Konstantin Shcheglov <[email protected]> Reviewed-by: Samuel Rawlins <[email protected]>
1 parent a2ff602 commit 238cd4c

File tree

32 files changed

+249
-136
lines changed

32 files changed

+249
-136
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ class CiderCompletionComputer {
172172
var suggestionBuilders = <CompletionSuggestionBuilder>[];
173173
var importedLibraries =
174174
target.withEnclosing2
175-
.expand((fragment) => fragment.libraryImports2)
175+
.expand((fragment) => fragment.libraryImports)
176176
.map((import) => import.importedLibrary)
177177
.nonNulls
178178
.toSet();

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -257,7 +257,7 @@ class CheckNameResponse {
257257
}
258258
} else if (element is MockLibraryImportElement) {
259259
var unit = (await canRename._fileResolver.resolve(path: sourcePath)).unit;
260-
var index = element.libraryFragment.libraryImports2.indexOf(
260+
var index = element.libraryFragment.libraryImports.indexOf(
261261
element.import,
262262
);
263263
var node = unit.directives.whereType<ImportDirective>().elementAt(index);

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,16 +18,16 @@ class ImplementedComputer {
1818
ImplementedComputer(this.searchEngine, this.unitElement);
1919

2020
Future<void> compute() async {
21-
for (var fragment in unitElement.classes2) {
21+
for (var fragment in unitElement.classes) {
2222
await _computeForInterfaceElement(fragment.element);
2323
}
24-
for (var fragment in unitElement.enums2) {
24+
for (var fragment in unitElement.enums) {
2525
await _computeForInterfaceElement(fragment.element);
2626
}
27-
for (var fragment in unitElement.extensionTypes2) {
27+
for (var fragment in unitElement.extensionTypes) {
2828
await _computeForInterfaceElement(fragment.element);
2929
}
30-
for (var fragment in unitElement.mixins2) {
30+
for (var fragment in unitElement.mixins) {
3131
await _computeForInterfaceElement(fragment.element);
3232
}
3333
}

pkg/analysis_server/lib/src/services/completion/dart/declaration_helper.dart

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -317,7 +317,7 @@ class DeclarationHelper {
317317

318318
void addImportPrefixes() {
319319
var library = request.libraryElement;
320-
for (var element in library.firstFragment.libraryImports2) {
320+
for (var element in library.firstFragment.libraryImports) {
321321
var importPrefix = element.prefix2;
322322
if (importPrefix == null) {
323323
continue;
@@ -940,7 +940,7 @@ class DeclarationHelper {
940940
// TODO(brianwilkerson): This will create suggestions for elements that
941941
// conflict with different elements imported from a different library. Not
942942
// sure whether that's the desired behavior.
943-
for (var importElement in library.firstFragment.libraryImports2) {
943+
for (var importElement in library.firstFragment.libraryImports) {
944944
var importedLibrary = importElement.importedLibrary;
945945
if (importedLibrary != null) {
946946
_addConstructorsImportedFrom(
@@ -958,7 +958,7 @@ class DeclarationHelper {
958958
// TODO(brianwilkerson): This will create suggestions for elements that
959959
// conflict with different elements imported from a different library. Not
960960
// sure whether that's the desired behavior.
961-
for (var importElement in library.firstFragment.libraryImports2) {
961+
for (var importElement in library.firstFragment.libraryImports) {
962962
var importedLibrary = importElement.importedLibrary;
963963
if (importedLibrary != null) {
964964
_addDeclarationsImportedFrom(

pkg/analysis_server/lib/src/services/completion/dart/not_imported_completion_pass.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -218,7 +218,7 @@ class _ImportSummary {
218218

219219
_ImportSummary(LibraryElement library) {
220220
for (var fragment in library.fragments) {
221-
for (var import in fragment.libraryImports2) {
221+
for (var import in fragment.libraryImports) {
222222
var importedLibrary = import.importedLibrary;
223223
if (importedLibrary != null) {
224224
if (import.combinators.isEmpty) {

pkg/analysis_server/lib/src/services/correction/dart/change_to.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@ class ChangeTo extends ResolvedCorrectionProducer {
136136
}
137137
// Check elements from imports.
138138
for (var importElement
139-
in unitResult.libraryElement2.firstFragment.libraryImports2) {
139+
in unitResult.libraryElement2.firstFragment.libraryImports) {
140140
if (importElement.prefix2?.element.name == prefixName) {
141141
var namespace = getImportNamespace(importElement);
142142
finder._updateList(namespace.values);
@@ -243,7 +243,7 @@ class ChangeTo extends ResolvedCorrectionProducer {
243243
}
244244
// Check unprefixed imports.
245245
for (var importElement
246-
in unitResult.libraryElement2.firstFragment.libraryImports2) {
246+
in unitResult.libraryElement2.firstFragment.libraryImports) {
247247
if (importElement.prefix2?.element.name == prefixName) {
248248
var namespace = getImportNamespace(importElement);
249249
finder._updateList(namespace.values);

pkg/analysis_server/lib/src/services/correction/dart/create_class.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -199,7 +199,7 @@ class _CreateClass extends ResolvedCorrectionProducer {
199199
filePath = file;
200200
prefix = '$eol$eol';
201201
} else {
202-
for (var import in libraryElement2.firstFragment.libraryImports2) {
202+
for (var import in libraryElement2.firstFragment.libraryImports) {
203203
if (_prefixElement is PrefixElement &&
204204
import.prefix2?.element == _prefixElement) {
205205
var library = import.importedLibrary;

pkg/analysis_server/lib/src/services/correction/dart/create_mixin.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,7 @@ class _CreateMixin extends ResolvedCorrectionProducer {
153153
filePath = file;
154154
prefix = '$eol$eol';
155155
} else {
156-
for (var import in libraryElement2.firstFragment.libraryImports2) {
156+
for (var import in libraryElement2.firstFragment.libraryImports) {
157157
if (prefixElement is PrefixElement &&
158158
import.prefix2?.element == prefixElement) {
159159
var library = import.importedLibrary;

pkg/analysis_server/lib/src/services/correction/dart/data_driven.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ class DataDriven extends MultiCorrectionProducer {
2727
var importedUris = <Uri>[];
2828
var library = unitResult.libraryElement2;
2929
var fragment = library.firstFragment;
30-
for (var importElement in fragment.libraryImports2) {
30+
for (var importElement in fragment.libraryImports) {
3131
// TODO(brianwilkerson): Filter based on combinators to help avoid making
3232
// invalid suggestions.
3333
var uri = importElement.uri;

pkg/analysis_server/lib/src/services/correction/dart/import_library.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -236,7 +236,7 @@ class ImportLibrary extends MultiCorrectionProducer {
236236
// Maybe there is an existing import, but it is with prefix and we don't use
237237
// this prefix.
238238
var alreadyImportedWithPrefix = <LibraryElement>{};
239-
for (var import in unitResult.libraryFragment.libraryImports2) {
239+
for (var import in unitResult.libraryFragment.libraryImports) {
240240
// Prepare the element.
241241
var libraryElement = import.importedLibrary;
242242
if (libraryElement == null) {
@@ -400,7 +400,7 @@ class ImportLibrary extends MultiCorrectionProducer {
400400
// additional analysis.
401401
var foundImport = false;
402402
var names = <_PrefixedName>[];
403-
for (var import in unitResult.libraryFragment.libraryImports2) {
403+
for (var import in unitResult.libraryFragment.libraryImports) {
404404
// prepare element
405405
var importedLibrary = import.importedLibrary;
406406
if (importedLibrary == null || importedLibrary != libraryToImport) {

0 commit comments

Comments
 (0)