SA1509 should ignore blank lines in dictionary initialization

[ArtemBan]

Sometimes in long dictionary initialization useful to group items by inserting blank lines between them for readability:

Dictionary<MyError, Guid> errorUids = new Dictionary<MyError, Guid>
{
    { new MyError { Name = "ErrorName1", Severity = Severity.Medium}, new Guid("xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx") },
    { new MyError { Name = "ErrorName2", Severity = Severity.Medium }, new Guid("xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx") },

    { new MyError { Name = "ErrorName3", Severity = Severity.Serious }, new Guid("xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx") },
    { new MyError { Name = "ErrorName4", Severity = Severity.Serious }, new Guid("xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx") },
    { new MyError { Name = "ErrorName5", Severity = Severity.Serious }, new Guid("xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx") },
    { new MyError { Name = "ErrorName6", Severity = Severity.Serious }, new Guid("xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx") },

    { new MyError { Name = "ErrorName7", Severity = Severity.Critical }, new Guid("xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx") },
    { new MyError { Name = "ErrorName8", Severity = Severity.Critical }, new Guid("xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx") },
    { new MyError { Name = "ErrorName9", Severity = Severity.Critical }, new Guid("xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx") },
};

Such code now causes warning, while the rule is obviously intended for code blocks and should ignore blank lines in initialization.

[sharwell]

💡 This can be mitigated today by adding a comment to each group.

_ = new Dictionary<int, int>
{
  { 1, 1 },

  // Description of group 2 
  { 2, 2 },

  // Description of group 3
  { 3, 3 },
};

💭 I would almost argue that this is a feature of this diagnostic. However, if we decide not to make a change to the rule we should at least document the case so users know how to address it.

[ArtemBan]

This can be mitigated today by adding a comment to each group.

In example I gave it's obvious that items are grouped by error severity. In similar cases comments will duplicate info already present in the code, which is bad style.