Even simpler

MikhailArkhipov · MikhailArkhipov · commit 96ae47905745 · 2018-10-05T13:07:09.000-07:00
diff --git a/src/Analysis/Engine/Impl/Values/Protocols.cs b/src/Analysis/Engine/Impl/Values/Protocols.cs
@@ -410,17 +410,12 @@ private void AppendParameterString(StringBuilder sb, AnalysisValue[] sets) {
                 return;
             }
 
-            if (sets.Length == 1) {
-                sb.Append(sets[0] is IHasQualifiedName qn ? qn.FullyQualifiedName : sets[0].ShortDescription);
-                return;
-            }
-
-            sb.Append('[');
+            sb.AppendIf(sets.Length > 1, "[");
             for (var i = 0; i < sets.Length; i++) {
                 sb.AppendIf(i > 0, ", ");
                 sb.Append(sets[i] is IHasQualifiedName qn ? qn.FullyQualifiedName : sets[i].ShortDescription);
             }
-            sb.Append(']');
+            sb.AppendIf(sets.Length > 1, "]");
         }
 
         protected override void EnsureMembers(IDictionary<string, IAnalysisSet> members) {

Original file line number	Diff line number	Diff line change
`@@ -410,17 +410,12 @@ private void AppendParameterString(StringBuilder sb, AnalysisValue[] sets) {`
`410`	`410`	`return;`
`411`	`411`	`}`
`412`	`412`
`413`		`- if (sets.Length == 1) {`
`414`		`- sb.Append(sets[0] is IHasQualifiedName qn ? qn.FullyQualifiedName : sets[0].ShortDescription);`
`415`		`- return;`
`416`		`- }`
`417`		`-`
`418`		`- sb.Append('[');`
	`413`	`+ sb.AppendIf(sets.Length > 1, "[");`
`419`	`414`	`for (var i = 0; i < sets.Length; i++) {`
`420`	`415`	`sb.AppendIf(i > 0, ", ");`
`421`	`416`	`sb.Append(sets[i] is IHasQualifiedName qn ? qn.FullyQualifiedName : sets[i].ShortDescription);`
`422`	`417`	`}`
`423`		`- sb.Append(']');`
	`418`	`+ sb.AppendIf(sets.Length > 1, "]");`
`424`	`419`	`}`
`425`	`420`
`426`	`421`	`protected override void EnsureMembers(IDictionary<string, IAnalysisSet> members) {`