Skip to content

Commit ee714c0

Browse files
committed
#3289 Apply code-review suggestions/comments (renaming method, improving conditions)
1 parent fc44991 commit ee714c0

2 files changed

Lines changed: 5 additions & 6 deletions

File tree

StyleCop.Analyzers/StyleCop.Analyzers.CodeFixes/ReadabilityRules/SA1130CodeFixProvider.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,7 @@ private static ImmutableArray<string> GetMethodInvocationArgumentList(SemanticMo
174174
var originalInvocableExpression = argumentListSyntax.Parent;
175175

176176
var originalSymbolInfo = semanticModel.GetSymbolInfo(originalInvocableExpression);
177-
var argumentIndex = SA1130UseLambdaSyntax.FindArgumentIndex(originalSymbolInfo, argumentSyntax, argumentListSyntax);
177+
var argumentIndex = SA1130UseLambdaSyntax.FindParameterIndex(originalSymbolInfo, argumentSyntax, argumentListSyntax);
178178
var parameterList = SA1130UseLambdaSyntax.GetDelegateParameterList(originalSymbolInfo.Symbol, argumentIndex);
179179
return parameterList.Parameters.Select(p => p.Identifier.ToString()).ToImmutableArray();
180180
}

StyleCop.Analyzers/StyleCop.Analyzers/ReadabilityRules/SA1130UseLambdaSyntax.cs

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -96,12 +96,11 @@ internal static ParameterListSyntax GetDelegateParameterList(ISymbol symbol, int
9696
return SyntaxFactory.ParameterList(SyntaxFactory.SeparatedList(syntaxParameters));
9797
}
9898

99-
internal static int FindArgumentIndex(SymbolInfo originalSymbolInfo, ArgumentSyntax argumentSyntax, BaseArgumentListSyntax argumentListSyntax)
99+
internal static int FindParameterIndex(SymbolInfo originalSymbolInfo, ArgumentSyntax argumentSyntax, BaseArgumentListSyntax argumentListSyntax)
100100
{
101101
// if delegate is passed as named argument of method try to find its position by argument name
102-
if (originalSymbolInfo.Symbol.Kind == SymbolKind.Method
103-
&& originalSymbolInfo.Symbol is IMethodSymbol methodSymbol
104-
&& argumentSyntax.NameColon != null)
102+
if (argumentSyntax.NameColon != null
103+
&& originalSymbolInfo.Symbol is IMethodSymbol methodSymbol)
105104
{
106105
var calledMethodParameters = methodSymbol.Parameters;
107106
var argumentIdentifier = argumentSyntax.NameColon.Name.Identifier.ValueText;
@@ -160,7 +159,7 @@ private static bool HandleMethodInvocation(SemanticModel semanticModel, Anonymou
160159
return false;
161160
}
162161

163-
var argumentIndex = FindArgumentIndex(originalSymbolInfo, argumentSyntax, argumentListSyntax);
162+
var argumentIndex = FindParameterIndex(originalSymbolInfo, argumentSyntax, argumentListSyntax);
164163

165164
// Determine the parameter list from the method that is invoked, as delegates without parameters are allowed, but they cannot be replaced by a lambda without parameters.
166165
var parameterList = GetDelegateParameterList(originalSymbolInfo.Symbol, argumentIndex);

0 commit comments

Comments
 (0)