Skip to content

Commit 7eea262

Browse files
scheglovCommit Queue
authored andcommitted
Deprecate NamedType.name2, use 'name' instead.
Change-Id: I27725e1530c60a6aa2e197b97e027de1238c964a Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/427980 Commit-Queue: Konstantin Shcheglov <[email protected]> Reviewed-by: Brian Wilkerson <[email protected]>
1 parent 7436b97 commit 7eea262

File tree

73 files changed

+300
-293
lines changed

Some content is hidden

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

73 files changed

+300
-293
lines changed

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1441,11 +1441,11 @@ class _DartUnitHighlightsComputerVisitor extends RecursiveAstVisitor<void> {
14411441

14421442
var type = node.type;
14431443
if (type != null) {
1444-
var isDynamic = type is DynamicType && node.name2.lexeme == 'dynamic';
1444+
var isDynamic = type is DynamicType && node.name.lexeme == 'dynamic';
14451445
var isNever = type is NeverType;
14461446
if (isDynamic || isNever) {
14471447
computer._addRegion_token(
1448-
node.name2,
1448+
node.name,
14491449
isDynamic
14501450
? HighlightRegionType.TYPE_NAME_DYNAMIC
14511451
: HighlightRegionType.CLASS,
@@ -1457,7 +1457,7 @@ class _DartUnitHighlightsComputerVisitor extends RecursiveAstVisitor<void> {
14571457

14581458
computer._addIdentifierRegion(
14591459
parent: node,
1460-
nameToken: node.name2,
1460+
nameToken: node.name,
14611461
element: node.element2,
14621462
);
14631463

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,7 @@ class DartUnitHoverComputer {
190190
ExtensionDeclaration() => node.name,
191191
FormalParameter() => node.name,
192192
MethodDeclaration() => node.name,
193-
NamedType() => node.name2,
193+
NamedType() => node.name,
194194
ConstructorDeclaration() => node.name ?? node.returnType,
195195
DeclaredIdentifier() => node.name,
196196
VariableDeclaration() => node.name,

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ class DartUnitOccurrencesComputerVisitor extends RecursiveAstVisitor<void> {
8383
if (node.name == null) {
8484
var element = node.element;
8585
if (element != null) {
86-
_addOccurrence(element, node.type.name2);
86+
_addOccurrence(element, node.type.name);
8787
}
8888
return; // skip visitNamedType.
8989
}
@@ -205,7 +205,7 @@ class DartUnitOccurrencesComputerVisitor extends RecursiveAstVisitor<void> {
205205
void visitNamedType(NamedType node) {
206206
var element = node.element2;
207207
if (element != null) {
208-
_addOccurrence(element, node.name2);
208+
_addOccurrence(element, node.name);
209209
}
210210

211211
super.visitNamedType(node);

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ mixin EditableArgumentsMixin {
5757

5858
String? widgetName, widgetDocumentation;
5959
if (invocation is InstanceCreationExpression) {
60-
widgetName = invocation.constructorName.type.name2.lexeme;
60+
widgetName = invocation.constructorName.type.name.lexeme;
6161

6262
if (invocation.constructorName.element case var element?) {
6363
widgetDocumentation = getDocumentation(result, element);

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ class TypeDefinitionHandler
8282
SyntacticEntity? originEntity;
8383
DartType? type;
8484
if (node is NamedType) {
85-
originEntity = node.name2;
85+
originEntity = node.name;
8686
var element = node.element2;
8787
if (element case analyzer.InterfaceElement element) {
8888
type = element.thisType;

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ class IdentifierHelper {
8080
/// Adds any suggestions for a variable with the given [type].
8181
void addVariable(TypeAnnotation? type) {
8282
if (type is NamedType) {
83-
addSuggestionsFromTypeName(type.name2.lexeme);
83+
addSuggestionsFromTypeName(type.name.lexeme);
8484
}
8585
}
8686

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

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1118,7 +1118,7 @@ class InScopeCompletionPass extends SimpleAstVisitor<void> {
11181118
if (offset <= extendsKeyword.end) {
11191119
keywordHelper.addKeyword(Keyword.EXTENDS);
11201120
} else if (node.superclass.isFullySynthetic ||
1121-
node.superclass.name2.coversOffset(offset)) {
1121+
node.superclass.name.coversOffset(offset)) {
11221122
collector.completionLocation = 'ExtendsClause_superclass';
11231123
_forTypeAnnotation(node, mustBeExtensible: true);
11241124
}
@@ -1401,7 +1401,7 @@ class InScopeCompletionPass extends SimpleAstVisitor<void> {
14011401
if (offset < variable.name.offset) {
14021402
var type = variable.type;
14031403
if (type == null ||
1404-
(type is NamedType && offset <= type.name2.end)) {
1404+
(type is NamedType && offset <= type.name.end)) {
14051405
_forTypeAnnotation(node);
14061406
}
14071407
}
@@ -1626,7 +1626,7 @@ class InScopeCompletionPass extends SimpleAstVisitor<void> {
16261626
if (namedType.end < offset) {
16271627
identifierHelper(
16281628
includePrivateIdentifiers: false,
1629-
).addSuggestionsFromTypeName(namedType.name2.lexeme);
1629+
).addSuggestionsFromTypeName(namedType.name.lexeme);
16301630
return;
16311631
}
16321632
}
@@ -1674,7 +1674,7 @@ class InScopeCompletionPass extends SimpleAstVisitor<void> {
16741674
@override
16751675
void visitImportPrefixReference(ImportPrefixReference node) {
16761676
var parent = node.parent;
1677-
if (parent is NamedType && offset <= parent.name2.offset) {
1677+
if (parent is NamedType && offset <= parent.name.offset) {
16781678
var element = node.element2;
16791679
DartType type;
16801680
collector.completionLocation = 'PropertyAccess_propertyName';
@@ -2582,7 +2582,7 @@ class InScopeCompletionPass extends SimpleAstVisitor<void> {
25822582
if (type.importPrefix case var importPrefix?) {
25832583
var prefixElement = importPrefix.element2;
25842584
if (prefixElement is PrefixElement) {
2585-
if (type.name2.coversOffset(offset)) {
2585+
if (type.name.coversOffset(offset)) {
25862586
declarationHelper(
25872587
mustBeType: true,
25882588
).addDeclarationsThroughImportPrefix(prefixElement);
@@ -2786,7 +2786,7 @@ class InScopeCompletionPass extends SimpleAstVisitor<void> {
27862786
if (namedType.end < offset) {
27872787
identifierHelper(
27882788
includePrivateIdentifiers: false,
2789-
).addSuggestionsFromTypeName(namedType.name2.lexeme);
2789+
).addSuggestionsFromTypeName(namedType.name.lexeme);
27902790
return;
27912791
}
27922792
}
@@ -3585,7 +3585,7 @@ class InScopeCompletionPass extends SimpleAstVisitor<void> {
35853585
if (node.type case NamedType namedType) {
35863586
identifierHelper(
35873587
includePrivateIdentifiers: false,
3588-
).addSuggestionsFromTypeName(namedType.name2.lexeme);
3588+
).addSuggestionsFromTypeName(namedType.name.lexeme);
35893589
}
35903590
}
35913591
} else if (parent is PatternField) {

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,7 @@ class AddDiagnosticPropertyReference extends ResolvedCorrectionProducer {
197197
var identifier = parameter.name;
198198
if (type is NamedType &&
199199
identifier != null &&
200-
type.name2.lexeme == 'DiagnosticPropertiesBuilder') {
200+
type.name.lexeme == 'DiagnosticPropertiesBuilder') {
201201
return identifier.lexeme;
202202
}
203203
}

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ class ChangeTo extends ResolvedCorrectionProducer {
114114
Token? nameToken;
115115
if (node is NamedType) {
116116
prefixName = node.importPrefix?.name.lexeme;
117-
nameToken = node.name2;
117+
nameToken = node.name;
118118
} else if (node is PrefixedIdentifier &&
119119
node.parent is NamedType &&
120120
node.prefix.element is PrefixElement) {

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
@@ -54,7 +54,7 @@ class CreateClass extends ResolvedCorrectionProducer {
5454
return;
5555
}
5656
}
57-
className = targetNode.name2.lexeme;
57+
className = targetNode.name.lexeme;
5858
requiresConstConstructor |= _requiresConstConstructor(targetNode);
5959
} else if (targetNode case SimpleIdentifier(
6060
:var parent,

0 commit comments

Comments
 (0)