dotnet/roslyn

Change signature can misplace comments, whitespace

Open

#33,801 opened on Mar 1, 2019

View on GitHub
 (4 comments) (0 reactions) (0 assignees)C# (20,414 stars) (4,257 forks)batch import
Area-IDEBugIDE-CodeStylehelp wanted

Description

Version Used: 2019 Preview 4

Steps to Reproduce:

    class Program
    {
        static void F(int a, int b, int c, int d)
        {
        }

        static void Main(string[] args)
        {
            F(1,
              2,// 2
              3,
              4);
        }
    }

Remove parameter int a using Change Signature.

Expected Behavior: Comments stay with their logically related argument/parameter

Actual Behavior:

Comments can end up in unexpected locations, sometimes logically related to the wrong entry. Indentation is not preserved.

namespace ConsoleApp3
{
    class Program
    {
        static void F(int b, int c, int d)
        {
        }

        static void Main(string[] args)
        {
            F(2,
                3,// 2
                4);
        }
    }
}

Contributor guide