Formatting changes from PR feedback

src/Features/CSharp/Portable/ConvertTupleToStruct/CSharpConvertTupleToStructCodeRefactoringProvider.cs

@@ -36,9 +36,7 @@ public CSharpConvertTupleToStructCodeRefactoringProvider()
         }
 
         protected override ArgumentSyntax GetArgumentWithChangedName(ArgumentSyntax argument, string name)
-        {
-            return argument.WithNameColon(ChangeName(argument.NameColon, name));
-        }
+            => argument.WithNameColon(ChangeName(argument.NameColon, name));
 
         private static NameColonSyntax? ChangeName(NameColonSyntax? nameColon, string name)
         {

src/Features/Core/Portable/ConvertTupleToStruct/AbstractConvertTupleToStructCodeRefactoringProvider.cs

@@ -53,6 +53,8 @@ internal abstract partial class AbstractConvertTupleToStructCodeRefactoringProvi
         where TTypeBlockSyntax : SyntaxNode
         where TNamespaceDeclarationSyntax : SyntaxNode
     {
+        protected abstract TArgumentSyntax GetArgumentWithChangedName(TArgumentSyntax argument, string name);
+
         public override async Task ComputeRefactoringsAsync(CodeRefactoringContext context)
         {
             var (document, textSpan, cancellationToken) = context;
@@ -305,7 +307,7 @@ private static string GetTitle(Scope scope)
                     // We should only ever get a default array (meaning, update the root), or a
                     // non-empty array.  We should never be asked to update exactly '0' nodes.
                     Debug.Assert(documentToUpdate.NodesToUpdate.IsDefault ||
-                                !documentToUpdate.NodesToUpdate.IsEmpty);
+                                 !documentToUpdate.NodesToUpdate.IsEmpty);
 
                     // If we were given specific nodes to update, only update those.  Otherwise
                     // updated everything from the root down.
@@ -679,8 +681,6 @@ private SyntaxNodeOrToken ConvertArgumentOrToken(SyntaxGenerator generator, Synt
             return (TArgumentSyntax)generator.Argument(expr).WithTriviaFrom(argument);
         }
 
-        protected abstract TArgumentSyntax GetArgumentWithChangedName(TArgumentSyntax argument, string name);
-
         private static async Task<bool> ReplaceMatchingTupleTypesAsync(
             Document document, SyntaxEditor editor, SyntaxNode startingNode,
             INamedTypeSymbol tupleType, TNameSyntax qualifiedTypeName,
@@ -879,7 +879,7 @@ private SyntaxNodeOrToken ConvertArgumentOrToken(SyntaxGenerator generator, Synt
         }
 
         private static string GetConstructorParameterName(string name)
-            => name.ToCamelCase(trimLeadingTypePrefix: false);
+            => name.ToCamelCase(trimLeadingTypePrefix: false); // TODO: This is the common case, but should ideally match the users style preference
 
         private class MyCodeAction : CodeAction.SolutionChangeAction
         {

src/Features/VisualBasic/Portable/ConvertTupleToStruct/VisualBasicConvertTupleToStructCodeRefactoringProvider.vb

@@ -40,14 +40,14 @@ Namespace Microsoft.CodeAnalysis.VisualBasic.ConvertTupleToStruct
 
             Dim nameColonEquals = simpleArgument.NameColonEquals
 
-            If nameColonEquals Is Nothing Then
-                Return argument
-            End If
-
             Return simpleArgument.WithNameColonEquals(ChangeName(nameColonEquals, name))
         End Function
 
         Private Shared Function ChangeName(nameColonEquals As NameColonEqualsSyntax, name As String) As NameColonEqualsSyntax
+            If nameColonEquals Is Nothing Then
+                Return Nothing
+            End If
+
             Dim newName = SyntaxFactory.IdentifierName(name).WithTriviaFrom(nameColonEquals.Name)
             Return nameColonEquals.WithName(newName)
         End Function