#136 - VB -> C# Use parentheses around single parameter in lambda when parameter has explicit type

Author: Nate Dahlquist

ICSharpCode.CodeConverter/CSharp/NodesVisitor.cs

@@ -1442,7 +1442,7 @@ public override CSharpSyntaxNode VisitSingleLineLambdaExpression(VBSyntax.Single
                     body = node.Body.Accept(TriviaConvertingVisitor);
                 }
                 var param = (ParameterListSyntax)node.SubOrFunctionHeader.ParameterList.Accept(TriviaConvertingVisitor);
-                if (param.Parameters.Count == 1)
+                if (param.Parameters.Count == 1 && param.Parameters.Single().Type == null)
                     return SyntaxFactory.SimpleLambdaExpression(param.Parameters[0], body);
                 return SyntaxFactory.ParenthesizedLambdaExpression(param, body);
             }
@@ -1451,7 +1451,7 @@ public override CSharpSyntaxNode VisitMultiLineLambdaExpression(VBSyntax.MultiLi
             {
                 var body = SyntaxFactory.Block(node.Statements.SelectMany(s => s.Accept(CreateMethodBodyVisitor())));
                 var param = (ParameterListSyntax)node.SubOrFunctionHeader.ParameterList.Accept(TriviaConvertingVisitor);
-                if (param.Parameters.Count == 1)
+                if (param.Parameters.Count == 1 && param.Parameters.Single().Type == null)
                     return SyntaxFactory.SimpleLambdaExpression(param.Parameters[0], body);
                 return SyntaxFactory.ParenthesizedLambdaExpression(param, body);
             }

Tests/CSharp/ExpressionTests.cs

@@ -550,7 +550,7 @@ class TestClass
 {
     private void TestMethod()
     {
-        Func<int, int> test = int a => a * 2;
+        Func<int, int> test = (int a) => a * 2;
         test(3);
     }
 }");