Skip to content

Commit decae83

Browse files
scheglovCommit Queue
authored andcommitted
Elements. Remove FieldFormalParameterElementOrMember and SuperFormalParameterElementOrMember.
Change-Id: I5af15189597d4f77a6637caa3646c4d5dde9970c Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/437500 Commit-Queue: Konstantin Shcheglov <[email protected]> Reviewed-by: Paul Berry <[email protected]>
1 parent 9975ed3 commit decae83

File tree

2 files changed

+14
-46
lines changed

2 files changed

+14
-46
lines changed

pkg/analyzer/lib/src/dart/element/element.dart

Lines changed: 11 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -2982,18 +2982,11 @@ class FieldFormalParameterElementImpl extends FormalParameterElementImpl
29822982
}
29832983
}
29842984

2985-
abstract class FieldFormalParameterElementOrMember
2986-
implements ParameterElementMixin {
2985+
class FieldFormalParameterFragmentImpl extends FormalParameterFragmentImpl
2986+
implements FieldFormalParameterFragment {
29872987
/// The field element associated with this field formal parameter, or `null`
29882988
/// if the parameter references a field that doesn't exist.
2989-
FieldElementOrMember? get field;
2990-
}
2991-
2992-
class FieldFormalParameterFragmentImpl extends FormalParameterFragmentImpl
2993-
implements
2994-
FieldFormalParameterElementOrMember,
2995-
FieldFormalParameterFragment {
2996-
@override
2989+
// TODO(scheglov): move to element
29972990
FieldFragmentImpl? field;
29982991

29992992
/// Initialize a newly created parameter element to have the given [name] and
@@ -9288,21 +9281,8 @@ class SuperFormalParameterElementImpl extends FormalParameterElementImpl
92889281
}
92899282
}
92909283

9291-
abstract class SuperFormalParameterElementOrMember
9292-
implements ParameterElementMixin {
9293-
/// The associated super-constructor parameter, from the super-constructor
9294-
/// that is referenced by the implicit or explicit super-constructor
9295-
/// invocation.
9296-
///
9297-
/// Can be `null` for erroneous code - not existing super-constructor,
9298-
/// no corresponding parameter in the super-constructor.
9299-
FormalParameterElementMixin? get superConstructorParameter;
9300-
}
9301-
93029284
class SuperFormalParameterFragmentImpl extends FormalParameterFragmentImpl
9303-
implements
9304-
SuperFormalParameterElementOrMember,
9305-
SuperFormalParameterFragment {
9285+
implements SuperFormalParameterFragment {
93069286
/// Initialize a newly created parameter element to have the given [name] and
93079287
/// [nameOffset].
93089288
SuperFormalParameterFragmentImpl({
@@ -9332,7 +9312,13 @@ class SuperFormalParameterFragmentImpl extends FormalParameterFragmentImpl
93329312
SuperFormalParameterFragmentImpl? get previousFragment =>
93339313
super.previousFragment as SuperFormalParameterFragmentImpl?;
93349314

9335-
@override
9315+
/// The associated super-constructor parameter, from the super-constructor
9316+
/// that is referenced by the implicit or explicit super-constructor
9317+
/// invocation.
9318+
///
9319+
/// Can be `null` for erroneous code - not existing super-constructor,
9320+
/// no corresponding parameter in the super-constructor.
9321+
// TODO(scheglov): move to element
93369322
FormalParameterElementMixin? get superConstructorParameter {
93379323
var enclosingElement = enclosingElement3;
93389324
if (enclosingElement is ConstructorFragmentImpl) {

pkg/analyzer/lib/src/dart/element/member.dart

Lines changed: 3 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -392,9 +392,7 @@ abstract class ExecutableMember extends Member
392392
/// A parameter element defined in a parameterized type where the values of the
393393
/// type parameters are known.
394394
class FieldFormalParameterMember extends ParameterMember
395-
implements
396-
FieldFormalParameterElementOrMember,
397-
FieldFormalParameterElement {
395+
implements FieldFormalParameterElement {
398396
factory FieldFormalParameterMember({
399397
required FieldFormalParameterFragmentImpl declaration,
400398
required MapSubstitution substitution,
@@ -425,16 +423,6 @@ class FieldFormalParameterMember extends ParameterMember
425423
return _declaration as FieldFormalParameterFragmentImpl;
426424
}
427425

428-
@override
429-
FieldElementOrMember? get field {
430-
var field = declaration.field;
431-
if (field == null) {
432-
return null;
433-
}
434-
435-
return FieldMember(declaration: field, substitution: substitution);
436-
}
437-
438426
@override
439427
FieldElement? get field2 {
440428
var field = baseElement.field2;
@@ -1352,9 +1340,7 @@ class SetterMember extends PropertyAccessorMember
13521340
}
13531341

13541342
class SuperFormalParameterMember extends ParameterMember
1355-
implements
1356-
SuperFormalParameterElementOrMember,
1357-
SuperFormalParameterElement {
1343+
implements SuperFormalParameterElement {
13581344
factory SuperFormalParameterMember({
13591345
required SuperFormalParameterFragmentImpl declaration,
13601346
required MapSubstitution substitution,
@@ -1400,18 +1386,14 @@ class SuperFormalParameterMember extends ParameterMember
14001386
bool get isCovariant => declaration.isCovariant;
14011387

14021388
@override
1403-
FormalParameterElementMixin? get superConstructorParameter {
1389+
FormalParameterElementMixin? get superConstructorParameter2 {
14041390
var superConstructorParameter = declaration.superConstructorParameter;
14051391
if (superConstructorParameter == null) {
14061392
return null;
14071393
}
14081394

14091395
return ParameterMember.from2(superConstructorParameter, substitution);
14101396
}
1411-
1412-
@override
1413-
FormalParameterElement? get superConstructorParameter2 =>
1414-
superConstructorParameter;
14151397
}
14161398

14171399
/// A variable element defined in a parameterized type where the values of the

0 commit comments

Comments
 (0)