Fix #188

src/dfmt/formatter.d

@@ -671,15 +671,21 @@ private:
                 currentLineLength = 0;
                 justAddedExtraNewline = true;
             }
-            if (config.dfmt_brace_style != BraceStyle.allman && currentIs(tok!"else"))
+            if (config.dfmt_brace_style == BraceStyle.otbs && peekIs(tok!"else"))
-                write(" ");
-            if (!peekIs(tok!",") && !peekIs(tok!")") && !peekIs(tok!";") && !peekIs(tok!"{"))
             {
+                write(" ");
                 index++;
-                newline();
             }
             else
-                index++;
+            {
+                if (!peekIs(tok!",") && !peekIs(tok!")") && !peekIs(tok!";") && !peekIs(tok!"{"))
+                {
+                    index++;
+                    newline();
+                }
+                else
+                    index++;
+            }
         }
     }
 

tests/otbs/issue0050.d.ref

@@ -6,8 +6,7 @@ void fun() {
                     doStuff();
                 else
                     morestuff();
-            }
+            } else
-            else
                 doStuff();
 
     cast(structalign_t) 1;

tests/otbs/issue0051.d.ref

@@ -1,7 +1,6 @@
 void f() {
     if (a) {
-    }
+    } else // wat
-    else // wat
     {
         if (!is_temp_arg_ref) {
             if (global.params.isOSX)

tests/otbs/issue0060.d.ref

@@ -1,6 +1,4 @@
 static if (a) {
-}
+} else static if (b) {
-else static if (b) {
+} else {
-}
-else {
 }

tests/otbs/issue0068.d.ref

@@ -1,4 +1,3 @@
 version (all) {
-}
+} else version (none) {
-else version (none) {
 }

tests/otbs/issue0081.d.ref

@@ -1,10 +1,8 @@
 unittest {
     if (0)
         if (0) {
-        }
+        } else if (0) {
-        else if (0) {
+        } else {
-        }
-        else {
         }
     doSomething();
 }

tests/otbs/issue0086.d.ref

@@ -12,8 +12,7 @@ void indent() {
         foreach (i; 0 .. indentLevel + tempIndent) {
             currentLineLength += config.tabSize;
             output.put("\t");
-        }
+        } else
-    else
         foreach (i; 0 .. indentLevel + tempIndent)
             foreach (j; 0 .. config.indentSize) {
                 output.put(" ");

tests/otbs/issue0093.d.ref

@@ -1,8 +1,7 @@
 unittest {
     if (x) {
         version (none) {
-        }
+        } else {
-        else {
         }
     }
 }

tests/otbs/issue0096.d.ref

@@ -8,7 +8,6 @@ version (Windows) {
 }
 version (Windows) {
     void func();
-}
+} else {
-else {
     void func();
 }

tests/otbs/issue0099.d.ref

@@ -9,8 +9,7 @@ unittest {
                 else {
                     excessivelyLongFunctionName(false);
                 }
-            }
+            } else
-            else
                 a();
     }
 }

tests/otbs/issue0169.d.ref

@@ -3,8 +3,7 @@ unittest {
         if (true) // comment
         {
 
-        }
+        } else {
-        else {
 
         }
     }

tests/otbs/issue0186.d.ref

@@ -12,12 +12,10 @@ void main() {
             return true;
         }()) {
 
-        }
+        } else {
-        else {
 
         }
-    }
+    } else {
-    else {
 
     }
 }