Fix #248

src/dfmt/formatter.d

@@ -595,7 +595,8 @@ private:
         }
         else if (index < tokens.length && (currentIs(tok!"@")
                 || isBasicType(tokens[index].type)
-                || currentIs(tok!"identifier") || currentIs(tok!"if")))
+                || currentIs(tok!"identifier") || currentIs(tok!"if"))
+                && !currentIsIndentedTemplateConstraint())
             write(" ");
     }
 
@@ -1021,6 +1022,7 @@ private:
                 else
                 {
                     writeToken();
+                    if (!currentIsIndentedTemplateConstraint())
                         write(" ");
                 }
             }
@@ -1030,6 +1032,14 @@ private:
         }
     }
 
+    bool currentIsIndentedTemplateConstraint()
+    {
+        return index < tokens.length
+            && astInformation.constraintLocations.canFindIndex(current.index)
+            && (config.dfmt_template_constraint_style == TemplateConstraintStyle.always_newline
+                || currentLineLength >= config.dfmt_soft_max_line_length);
+    }
+
     void formatOperator()
     {
         import std.algorithm : canFind;

tests/allman/issue0248.d.ref

@@ -0,0 +1,11 @@
+T[] arrayOp(T, Args...)(T res, Filter!(isType, Args) args) @trusted @nogc pure @things
+        if (Args[$ - 1] != "=")
+{
+
+}
+
+T[] arrayOp(T, Args...)(T res, Filter!(isType, Args) args) @trusted @nogc pure nothrow
+        if (Args[$ - 1] != "=")
+{
+
+}

tests/issue0248.d

@@ -0,0 +1,11 @@
+T[] arrayOp(T, Args...)(T res, Filter!(isType, Args) args) @trusted @nogc pure @things
+		if (Args[$ - 1] != "=")
+{
+
+}
+
+T[] arrayOp(T, Args...)(T res, Filter!(isType, Args) args) @trusted @nogc pure nothrow
+		if (Args[$ - 1] != "=")
+{
+
+}

tests/otbs/issue0248.d.ref

@@ -0,0 +1,9 @@
+T[] arrayOp(T, Args...)(T res, Filter!(isType, Args) args) @trusted @nogc pure @things
+        if (Args[$ - 1] != "=") {
+
+}
+
+T[] arrayOp(T, Args...)(T res, Filter!(isType, Args) args) @trusted @nogc pure nothrow
+        if (Args[$ - 1] != "=") {
+
+}