FormatOps: fix an overly aggressive optimal token

Author: kitbellew

scalafmt-core/shared/src/main/scala/org/scalafmt/internal/FormatOps.scala

@@ -153,7 +153,8 @@ class FormatOps(
         case t if t.end >= end => Some(start.left)
         case _ if start.hasBlankLine => Some(start.left)
         case _: T.RightParen
-            if start.left.is[T.RightParen] || start.left.is[T.LeftParen] =>
+            if start.left.is[T.LeftParen] || start.left.is[T.RightParen] &&
+              !start.meta.rightOwner.is[Member.SyntaxValuesClause] =>
           None
         case _: T.RightBracket if start.left.is[T.RightBracket] => None
         case _: T.Comma | _: T.LeftParen | _: T.Semicolon | _: T.RightArrow |

scalafmt-tests/src/test/resources/newlines/source_fold.stat

@@ -7034,8 +7034,7 @@ class a {
   )(in: Input[E]): Iteratee[E, Iteratee[E, A]] = in match {
     case Input.El(e) => Iteratee.flatten(
         Future(p(e))(pec).map(b =>
-          if (b) Done(inner, in)
-          else stepNoBreak(inner)(in)
+          if (b) Done(inner, in) else stepNoBreak(inner)(in)
         )(dec)
       )
     case _ => stepNoBreak(inner)(in)
@@ -7051,9 +7050,9 @@ class a {
 }
 >>>
 class a {
-  atPos(body.pos)(casegen.one(substitution(
-    body
-  ))) // since SubstOnly treemakers are dropped, need to do it here
+  atPos(body.pos)(casegen.one(
+    substitution(body)
+  )) // since SubstOnly treemakers are dropped, need to do it here
 }
 <<< SM 7.4.0: 52
 maxColumn = 76