#464 Incorrect indentation of C# collection initialization

[a9892b58]

Source:

List<int> ints = new List<int>(count)
{
    MyClass.GetInt(0),
    MyClass.GetInt(2)
};

Current trunk:

List<int> ints = new List<int>(count)
{
    MyClass.GetInt(0),
                   MyClass.GetInt(2)
};

I attach a patch with a workaround I did locally to fix the issue.

[a9892b58]

I attach an improved patch, that does not fail any unit tests. Also a patch adding new unit test.

[Francois Botha]

Before I log a new issue, I want to check whether what I'm experiencing is related to this issue.

I have this code and AStyle config:

   public class Sample
   {
      private static MyObject myObject = Initialize(DateTime.Today,
                                                    new List<double>() { 0.5, 0.7 },
                                                    String.Empty);
   }

Notice that the parameters of Initialize are aligned.

# braces and indent
style=allman
indent=spaces=3

# indentation
min-conditional-indent=0
max-continuation-indent=80
indent-classes
indent-namespaces
indent-switches

# padding
pad-oper
pad-header
unpad-paren
align-pointer=type

# formatting
break-one-line-headers
keep-one-line-blocks
keep-one-line-statements
convert-tabs

# other options
suffix=none
exclude=obj

I expect the code to remain unchanged, but the List<double> initializer messes something up. Output:

   public class Sample
   {
      private static MyObject myObject = Initialize(DateTime.Today,
      new List<double>() { 0.5, 0.7 },
      String.Empty);
   }

[André Simon]

The first issue was fixed in 3.4.6