Skip to content

Commit 73cfad0

Browse files
committed
Switch to deterministic parameter
Use the first parameter for inferring the length as intended by the code
1 parent f6633ac commit 73cfad0

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

MKL.NET.WrapperGenerator/WrapperGenerator.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,10 @@ public void Initialize(GeneratorInitializationContext context)
3636
return candidates[0];
3737
}
3838

39-
private static (ISet<ParameterSyntax> changed, ParameterListSyntax newList)
39+
private static (IList<ParameterSyntax> changed, ParameterListSyntax newList)
4040
TransformParameters(MethodDeclarationSyntax mds, Func<ParameterSyntax, ParameterSyntax?> f)
4141
{
42-
var changed = mds.ParameterList.Parameters.Select(ps => f(ps) is null ? null : ps).Where(ps => ps != null).ToImmutableHashSet();
42+
var changed = mds.ParameterList.Parameters.Select(ps => f(ps) is null ? null : ps).Where(ps => ps != null).ToImmutableList();
4343
var newList = SyntaxFactory.ParameterList(SyntaxFactory.SeparatedList(mds.ParameterList.Parameters.Select(ps => f(ps) ?? ps)));
4444

4545
return (changed!, newList);
@@ -57,7 +57,7 @@ private enum AdditionalTransformation
5757

5858
void WriterTransformedMethod(
5959
MethodDeclarationSyntax mds, ClassDeclarationSyntax nativeCds,
60-
(ISet<ParameterSyntax> changed, ParameterListSyntax newList) transformation, StringBuilder sb,
60+
(IList<ParameterSyntax> changed, ParameterListSyntax newList) transformation, StringBuilder sb,
6161
AdditionalTransformation trafo)
6262
{
6363
(ParameterSyntax lengthParam, string takeLengthFrom)? lengthOptions = null;

0 commit comments

Comments
 (0)