Skip to content

Commit f47cc44

Browse files
fshcheglovCommit Queue
authored andcommitted
Remove Xyz2 suffix getters from /analysis
Change-Id: I10221b86f63ee8a2d74938ef0438945b43e11f6b Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/441990 Reviewed-by: Konstantin Shcheglov <[email protected]> Reviewed-by: Brian Wilkerson <[email protected]> Commit-Queue: Paul Berry <[email protected]> Reviewed-by: Paul Berry <[email protected]>
1 parent 0687036 commit f47cc44

File tree

64 files changed

+241
-156
lines changed

Some content is hidden

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

64 files changed

+241
-156
lines changed

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ class ImportElementsComputer {
3737
var filteredImportedElements = _filterImportedElements(
3838
importedElementsList,
3939
);
40-
var libraryElement = libraryResult.libraryElement2;
40+
var libraryElement = libraryResult.libraryElement;
4141
var uriConverter = libraryResult.session.uriConverter;
4242
var existingImports = <ImportDirective>[];
4343
for (var directive in unit.directives) {
@@ -358,7 +358,7 @@ class ImportElementsComputer {
358358
}
359359

360360
bool _hasElement(String prefix, String name) {
361-
var scope = libraryResult.libraryElement2.firstFragment.scope;
361+
var scope = libraryResult.libraryElement.firstFragment.scope;
362362

363363
if (prefix.isNotEmpty) {
364364
var prefixElement = scope.lookup(prefix).getter;

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ class EditImportElementsHandler extends LegacyHandler {
4040
sendResponse(Response.importElementsInvalidFile(request));
4141
return;
4242
}
43-
var libraryUnit = result.libraryElement2.firstFragment;
43+
var libraryUnit = result.libraryElement.firstFragment;
4444
if (libraryUnit != result.libraryFragment) {
4545
// The file in the request is a part of a library. We need to pass the
4646
// defining compilation unit to the computer, not the part.

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ extension SomeResolvedLibraryResultExtension on SomeResolvedLibraryResult? {
115115
Version get effectiveLanguageVersion {
116116
var self = this;
117117
if (self is ResolvedLibraryResult) {
118-
return self.element2.languageVersion.effective;
118+
return self.element.languageVersion.effective;
119119
}
120120
return DartFormatter.latestLanguageVersion;
121121
}

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
@@ -138,7 +138,7 @@ class NotImportedCompletionPass {
138138
}
139139

140140
var library = request.libraryElement;
141-
var element = elementResult.element2;
141+
var element = elementResult.element;
142142
if (element == library) {
143143
// Don't suggest elements from the library in which completion is being
144144
// requested. They've already been suggested.

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -132,11 +132,11 @@ class ChangeTo extends ResolvedCorrectionProducer {
132132
);
133133
// Check elements of this library.
134134
if (prefixName == null) {
135-
finder._updateList(unitResult.libraryElement2.classes);
135+
finder._updateList(unitResult.libraryElement.classes);
136136
}
137137
// Check elements from imports.
138138
for (var importElement
139-
in unitResult.libraryElement2.firstFragment.libraryImports) {
139+
in unitResult.libraryElement.firstFragment.libraryImports) {
140140
if (importElement.prefix?.element.name == prefixName) {
141141
var namespace = getImportNamespace(importElement);
142142
finder._updateList(namespace.values);
@@ -237,13 +237,13 @@ class ChangeTo extends ResolvedCorrectionProducer {
237237
);
238238
// Check to this library units.
239239
if (prefixName == null) {
240-
for (var function in unitResult.libraryElement2.topLevelFunctions) {
240+
for (var function in unitResult.libraryElement.topLevelFunctions) {
241241
finder._update(function);
242242
}
243243
}
244244
// Check unprefixed imports.
245245
for (var importElement
246-
in unitResult.libraryElement2.firstFragment.libraryImports) {
246+
in unitResult.libraryElement.firstFragment.libraryImports) {
247247
if (importElement.prefix?.element.name == prefixName) {
248248
var namespace = getImportNamespace(importElement);
249249
finder._updateList(namespace.values);

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ class ConvertPartOfToUri extends ResolvedCorrectionProducer {
3333
}
3434

3535
var pathContext = resourceProvider.pathContext;
36-
var libraryFragment = unitResult.libraryElement2.firstFragment;
36+
var libraryFragment = unitResult.libraryElement.firstFragment;
3737
var libraryPath = libraryFragment.source.fullName;
3838
var partPath = unitResult.path;
3939
var relativePath = pathContext.relative(

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ class CreateFile extends ResolvedCorrectionProducer {
5151
if (source != null) {
5252
var pathContext = resourceProvider.pathContext;
5353
var relativePath = pathContext.relative(
54-
unitResult.libraryElement2.firstFragment.source.fullName,
54+
unitResult.libraryElement.firstFragment.source.fullName,
5555
from: pathContext.dirname(source.fullName),
5656
);
5757

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
@@ -25,7 +25,7 @@ class DataDriven extends MultiCorrectionProducer {
2525
@override
2626
Future<List<ResolvedCorrectionProducer>> get producers async {
2727
var importedUris = <Uri>[];
28-
var library = unitResult.libraryElement2;
28+
var library = unitResult.libraryElement;
2929
var fragment = library.firstFragment;
3030
for (var importElement in fragment.libraryImports) {
3131
// TODO(brianwilkerson): Filter based on combinators to help avoid making

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
@@ -506,7 +506,7 @@ class ImportLibrary extends MultiCorrectionProducer {
506506

507507
var names = <_PrefixedName>[];
508508
var name = Name.forLibrary(
509-
dartFixContext.unitResult.libraryElement2,
509+
dartFixContext.unitResult.libraryElement,
510510
memberName,
511511
);
512512
await for (var libraryToImport in librariesWithExtensions(name)) {
@@ -918,7 +918,7 @@ class _ImportLibraryCombinatorMultiple extends ResolvedCorrectionProducer {
918918
newCombinatorCode = ' ${keyword.lexeme} ${names.join(', ')}';
919919
}
920920
var libraryPath =
921-
unitResult.libraryElement2.firstFragment.source.fullName;
921+
unitResult.libraryElement.firstFragment.source.fullName;
922922
await builder.addDartFileEdit(libraryPath, (builder) {
923923
builder.addSimpleReplacement(
924924
SourceRange(offset - 1, length + 1),

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

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -103,10 +103,7 @@ class _UseDifferentDivisionOperator extends ResolvedCorrectionProducer {
103103
}
104104
// All extensions available in the current scope for the left operand that
105105
// define the other division operator.
106-
var name = Name.forLibrary(
107-
unitResult.libraryElement2,
108-
otherOperator.lexeme,
109-
);
106+
var name = Name.forLibrary(unitResult.libraryElement, otherOperator.lexeme);
110107
var hasNoExtensionWithOtherDivisionOperator =
111108
await librariesWithExtensions(name).where((library) {
112109
return library.exportedExtensions

0 commit comments

Comments
 (0)