Fixes: removing "<>" from "Characters Requiring Whitespace" breaks auto-formatting #975

elsassph · elsassph · commit f8f04831f106 · 2016-01-13T23:43:24.000Z
diff --git a/External/Plugins/ASCompletion/Completion/Reformater.cs b/External/Plugins/ASCompletion/Completion/Reformater.cs
@@ -233,7 +233,11 @@ public static string ReformatLine(string txt, ReformatOptions options, ref int o
                         needSpace = true;
                         continue;
                     }
-                    else needSpace = (c != '!' || (c2 != '(' && c2 != '['));
+                    else if (c != '!' || (c2 != '(' && c2 != '[')) 
+                    {
+                        if (options.Operators.IndexOf(c2) >= 0 && options.Operators.IndexOf(c) >= 0) needSpace = false;
+                        else needSpace = (c != '!' || (c2 != '(' && c2 != '['));
+                    }
 
                     if (i < n)
                     {

Original file line number	Diff line number	Diff line change
`@@ -233,7 +233,11 @@ public static string ReformatLine(string txt, ReformatOptions options, ref int o`
`233`	`233`	`needSpace = true;`
`234`	`234`	`continue;`
`235`	`235`	`}`
`236`		`- else needSpace = (c != '!' \|\| (c2 != '(' && c2 != '['));`
	`236`	`+ else if (c != '!' \|\| (c2 != '(' && c2 != '['))`
	`237`	`+ {`
	`238`	`+ if (options.Operators.IndexOf(c2) >= 0 && options.Operators.IndexOf(c) >= 0) needSpace = false;`
	`239`	`+ else needSpace = (c != '!' \|\| (c2 != '(' && c2 != '['));`
	`240`	`+ }`
`237`	`241`
`238`	`242`	`if (i < n)`
`239`	`243`	`{`