Skip to content

Commit 6a5c577

Browse files
fshcheglovCommit Queue
authored andcommitted
Rename / deprecate more getters and methods.
* Deprecate `DirectiveUriWithLibrary.library2`, use `library` instead. * Deprecate `ElementAnnotation.element2`, use `element` instead. * Deprecate `LibraryImport.prefix2`, use `prefix` instead. * Deprecate `MultiplyDefinedElement.conflictingElements2`, use `conflictingElements` instead. * Deprecate `ScopeLookupResult.getter2`, use `getter` instead. * Deprecate `ScopeLookupResult.setter2`, use `setter` instead. Change-Id: I523eb21d194b661eed5bcbbfc923e9a7de87c962 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/440361 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 df9267d commit 6a5c577

File tree

70 files changed

+281
-214
lines changed

Some content is hidden

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

70 files changed

+281
-214
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
@@ -361,7 +361,7 @@ class ImportElementsComputer {
361361
var scope = libraryResult.libraryElement2.firstFragment.scope;
362362

363363
if (prefix.isNotEmpty) {
364-
var prefixElement = scope.lookup(prefix).getter2;
364+
var prefixElement = scope.lookup(prefix).getter;
365365
if (prefixElement is PrefixElement) {
366366
scope = prefixElement.scope;
367367
} else {
@@ -370,7 +370,7 @@ class ImportElementsComputer {
370370
}
371371

372372
var lookupResult = scope.lookup(name);
373-
return lookupResult.getter2 != null || lookupResult.setter2 != null;
373+
return lookupResult.getter != null || lookupResult.setter != null;
374374
}
375375

376376
/// Return `true` if the given [import] matches the given specification of

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
@@ -149,7 +149,7 @@ class ImportsHandler
149149

150150
var isMatch =
151151
element is MultiplyDefinedElement
152-
? element.conflictingElements2.contains(importedElement)
152+
? element.conflictingElements.contains(importedElement)
153153
: element == importedElement;
154154

155155
if (isMatch) {

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -211,7 +211,7 @@ class DeclarationHelper {
211211
prefix: null,
212212
);
213213

214-
if (importElement.prefix2 case var importPrefix?) {
214+
if (importElement.prefix case var importPrefix?) {
215215
if (importPrefix.isDeferred) {
216216
var matcherScore = state.matcher.score('loadLibrary');
217217
if (matcherScore != -1) {
@@ -318,7 +318,7 @@ class DeclarationHelper {
318318
void addImportPrefixes() {
319319
var library = request.libraryElement;
320320
for (var element in library.firstFragment.libraryImports) {
321-
var importPrefix = element.prefix2;
321+
var importPrefix = element.prefix;
322322
if (importPrefix == null) {
323323
continue;
324324
}
@@ -946,7 +946,7 @@ class DeclarationHelper {
946946
_addConstructorsImportedFrom(
947947
library: importedLibrary,
948948
namespace: importElement.namespace,
949-
prefix: importElement.prefix2?.element.name,
949+
prefix: importElement.prefix?.element.name,
950950
);
951951
}
952952
}
@@ -964,7 +964,7 @@ class DeclarationHelper {
964964
_addDeclarationsImportedFrom(
965965
library: importedLibrary,
966966
namespace: importElement.namespace,
967-
prefix: importElement.prefix2?.element.name,
967+
prefix: importElement.prefix?.element.name,
968968
);
969969
if (importedLibrary.isDartCore && mustBeType) {
970970
var name = 'Never';

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ class AmbiguousImportFix extends MultiCorrectionProducer {
3737
if (element is! MultiplyDefinedElement) {
3838
return const [];
3939
}
40-
var conflictingElements = element.conflictingElements2;
40+
var conflictingElements = element.conflictingElements;
4141
var name = element.name;
4242
if (name == null || name.isEmpty) {
4343
return const [];

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
@@ -137,7 +137,7 @@ class ChangeTo extends ResolvedCorrectionProducer {
137137
// Check elements from imports.
138138
for (var importElement
139139
in unitResult.libraryElement2.firstFragment.libraryImports) {
140-
if (importElement.prefix2?.element.name == prefixName) {
140+
if (importElement.prefix?.element.name == prefixName) {
141141
var namespace = getImportNamespace(importElement);
142142
finder._updateList(namespace.values);
143143
}
@@ -244,7 +244,7 @@ class ChangeTo extends ResolvedCorrectionProducer {
244244
// Check unprefixed imports.
245245
for (var importElement
246246
in unitResult.libraryElement2.firstFragment.libraryImports) {
247-
if (importElement.prefix2?.element.name == prefixName) {
247+
if (importElement.prefix?.element.name == prefixName) {
248248
var namespace = getImportNamespace(importElement);
249249
finder._updateList(namespace.values);
250250
}

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
@@ -201,7 +201,7 @@ class _CreateClass extends ResolvedCorrectionProducer {
201201
} else {
202202
for (var import in libraryElement2.firstFragment.libraryImports) {
203203
if (_prefixElement is PrefixElement &&
204-
import.prefix2?.element == _prefixElement) {
204+
import.prefix?.element == _prefixElement) {
205205
var library = import.importedLibrary;
206206
if (library != null) {
207207
targetUnit = library.firstFragment;

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
@@ -155,7 +155,7 @@ class _CreateMixin extends ResolvedCorrectionProducer {
155155
} else {
156156
for (var import in libraryElement2.firstFragment.libraryImports) {
157157
if (prefixElement is PrefixElement &&
158-
import.prefix2?.element == prefixElement) {
158+
import.prefix?.element == prefixElement) {
159159
var library = import.importedLibrary;
160160
if (library != null) {
161161
targetUnit = library.firstFragment;

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
@@ -123,7 +123,7 @@ class ImportLibrary extends MultiCorrectionProducer {
123123
case var exportedName?)
124124
exportedName,
125125
];
126-
var importPrefix = import.prefix2?.element;
126+
var importPrefix = import.prefix?.element;
127127
var importCombinator = _ImportLibraryCombinator(
128128
uri,
129129
combinators,
@@ -255,7 +255,7 @@ class ImportLibrary extends MultiCorrectionProducer {
255255
if (!kinds.contains(element.kind)) {
256256
continue;
257257
}
258-
var importPrefix = import.prefix2?.element;
258+
var importPrefix = import.prefix?.element;
259259
// Maybe update a "show"/"hide" directive.
260260
var (
261261
combinatorProducer,

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ LibraryImport? _getImportElement(
5050
continue;
5151
}
5252
// required prefix
53-
var prefixElement = libraryImport.prefix2?.element;
53+
var prefixElement = libraryImport.prefix?.element;
5454
if (prefixElement == null) {
5555
continue;
5656
}

pkg/analysis_server/lib/src/services/refactoring/legacy/extract_method.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1888,7 +1888,7 @@ extension on LibraryElement {
18881888
// Ensure import.
18891889
var importElement = _getImportElement(element);
18901890
if (importElement != null) {
1891-
var prefix = importElement.prefix2?.element;
1891+
var prefix = importElement.prefix?.element;
18921892
if (prefix != null) {
18931893
sb.write(prefix.displayName);
18941894
sb.write('.');

0 commit comments

Comments
 (0)