Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Commit a9d162e

Browse files
bwilkersonCommit Bot
authored andcommitted
Revert "[Analyzer] Handle parameters that are both deprecated and required"
This reverts commit 9926189. Reason for revert: Breaks the flutter build Original change's description: > [Analyzer] Handle parameters that are both deprecated and required > > Do not produce a diagnostic at the call-site in this case. > > Change-Id: I994704f77261b8e801a242c963236385f519eaa8 > Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/241008 > Reviewed-by: Brian Wilkerson <brianwilkerson@google.com> > Commit-Queue: Brian Wilkerson <brianwilkerson@google.com> # Not skipping CQ checks because original CL landed > 1 day ago. Change-Id: I2c6cbdcf8a08a178d77c6f875488f54f95788063 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/241441 Reviewed-by: Brian Wilkerson <brianwilkerson@google.com> Reviewed-by: Konstantin Shcheglov <scheglov@google.com> Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
1 parent 909dbe5 commit a9d162e

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

pkg/analyzer/lib/src/error/deprecated_member_use_verifier.dart

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -146,10 +146,6 @@ class DeprecatedMemberUseVerifier {
146146
return;
147147
}
148148

149-
if (element is ParameterElement && element.isNotOptional) {
150-
return;
151-
}
152-
153149
var errorNode = node;
154150
var parent = node.parent;
155151
if (parent is AssignmentExpression && parent.leftHandSide == node) {
@@ -208,7 +204,11 @@ class DeprecatedMemberUseVerifier {
208204
_visitParametersAndArguments(
209205
element.parameters,
210206
arguments.arguments,
211-
_checkForDeprecated,
207+
(parameter, argument) {
208+
if (parameter.isOptional) {
209+
_checkForDeprecated(parameter, argument);
210+
}
211+
},
212212
);
213213
}
214214
}

0 commit comments

Comments
 (0)