Fix #35

src/dfmt/formatter.d

@@ -599,7 +599,7 @@ private:
 
     void formatSwitch()
     {
-        if (indents.topIs(tok!"with"))
+        while (indents.topIs(tok!"with"))
             indents.pop();
         indents.push(tok!"switch");
         writeToken(); // switch
@@ -847,15 +847,35 @@ private:
         case tok!"..":
         case tok!"%":
         binary:
-            if (linebreakHints.canFind(index) || peekIs(tok!"comment", false))
+            immutable bool isWrapToken = linebreakHints.canFind(index) || peekIs(tok!"comment", false);
+            if (config.dfmt_split_operator_at_line_end)
             {
-                pushWrapIndent();
+                if (isWrapToken)
-                newline();
+                {
+                    pushWrapIndent();
+                    write(" ");
+                    writeToken();
+                    newline();
+                }
+                else
+                {
+                    write(" ");
+                    writeToken();
+                    write(" ");
+                }
             }
             else
+            {
+                if (isWrapToken)
+                {
+                    pushWrapIndent();
+                    newline();
+                }
+                else
+                    write(" ");
+                writeToken();
                 write(" ");
-            writeToken();
+            }
-            write(" ");
             break;
         default:
             writeToken();

src/dfmt/main.d

@@ -16,7 +16,7 @@ else
     import dfmt.formatter : format;
     import std.path : buildPath, expandTilde;
     import dfmt.editorconfig : getConfigFor;
-    import std.getopt : getopt;
+    import std.getopt : getopt, GetOptException;
 
     int main(string[] args)
     {
@@ -28,10 +28,13 @@ else
         void handleBooleans(string option, string value)
         {
             import dfmt.editorconfig : OptionalBoolean;
-            import std.exception : enforce;
+            import std.exception : enforceEx;
-            enforce(value == "true" || value == "false", "Invalid argument");
+            enforceEx!GetOptException(value == "true" || value == "false", "Invalid argument");
             switch (option)
             {
+            case "align_switch_statements":
+                optConfig.dfmt_align_switch_statements = value == "true" ? OptionalBoolean.t : OptionalBoolean.f;
+                break;
             case "outdent_attributes":
                 optConfig.dfmt_outdent_attributes = value == "true" ? OptionalBoolean.t : OptionalBoolean.f;
                 break;
@@ -48,21 +51,29 @@ else
             }
         }
 
-        getopt(args,
+        try
-            "align_switch_statements", &optConfig.dfmt_align_switch_statements,
+        {
-            "brace_style", &optConfig.dfmt_brace_style,
+            getopt(args,
-            "end_of_line", &optConfig.end_of_line,
+                "align_switch_statements", &handleBooleans,
-            "help|h", &showHelp,
+                "brace_style", &optConfig.dfmt_brace_style,
-            "indent_size", &optConfig.indent_size,
+                "end_of_line", &optConfig.end_of_line,
-            "indent_style|t", &optConfig.indent_style,
+                "help|h", &showHelp,
-            "inplace", &inplace,
+                "indent_size", &optConfig.indent_size,
-            "max_line_length", &optConfig.max_line_length,
+                "indent_style|t", &optConfig.indent_style,
-            "max_line_length", &optConfig.max_line_length,
+                "inplace", &inplace,
-            "outdent_attributes", &handleBooleans,
+                "max_line_length", &optConfig.max_line_length,
-            "outdent_labels", &handleBooleans,
+                "max_line_length", &optConfig.max_line_length,
-            "space_after_cast", &handleBooleans,
+                "outdent_attributes", &handleBooleans,
-            "split_operator_at_line_end", &handleBooleans,
+                "outdent_labels", &handleBooleans,
-            "tab_width", &optConfig.tab_width);
+                "space_after_cast", &handleBooleans,
+                "split_operator_at_line_end", &handleBooleans,
+                "tab_width", &optConfig.tab_width);
+        }
+        catch (GetOptException e)
+        {
+            stderr.writeln(e.msg);
+            return 1;
+        }
 
         if (showHelp)
         {

tests/allman/issue0035.d.ref

@@ -0,0 +1,7 @@
+unittest
+{
+    if (some_very_long_expression && some_very_long_expression &&
+            some_very_long_expression && some_very_long_expression && some_very_long_expression)
+    {
+    }
+}

tests/issue0035.args

@@ -0,0 +1 @@
+--split_operator_at_line_end=true

tests/issue0035.d

@@ -0,0 +1,8 @@
+unittest
+{
+    if (some_very_long_expression && some_very_long_expression
+        && some_very_long_expression && some_very_long_expression
+        && some_very_long_expression)
+    {
+    }
+}

tests/otbs/issue0035.d.ref

@@ -0,0 +1,5 @@
+unittest {
+    if (some_very_long_expression && some_very_long_expression &&
+            some_very_long_expression && some_very_long_expression && some_very_long_expression) {
+    }
+}