RedundantParens: check for placeholder in Apply

scalameta · Feb 5, 2022 · 8e279bd · 8e279bd
1 parent 63e83e1
commit 8e279bd
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/scalafmt-core/shared/src/main/scala/org/scalafmt/rewrite/RedundantParens.scala b/scalafmt-core/shared/src/main/scala/org/scalafmt/rewrite/RedundantParens.scala
@@ -60,9 +60,9 @@ class RedundantParens(ftoks: FormatTokens) extends FormatTokensRewrite.Rule {
 
       case _: Lit | _: Name | _: Term.Interpolate => true
 
-      case Term.Apply(_, List(b @ (_: Term.Block | _: Term.PartialFunction)))
+      case Term.Apply(f, List(b @ (_: Term.Block | _: Term.PartialFunction)))
           if b.tokens.headOption.exists(_.is[Token.LeftBrace]) =>
-        true
+        !RewriteCtx.hasPlaceholder(f)
 
       case _: Term.PartialFunction => true
 

diff --git a/scalafmt-tests/src/test/resources/rewrite/RedundantParens.stat b/scalafmt-tests/src/test/resources/rewrite/RedundantParens.stat
@@ -776,8 +776,6 @@ foo >>= (_.http(registry, port).map(Option.apply(_)).catchSome {
     bar
 })
 >>>
-foo >>= _.http(registry, port)
-  .map(Option.apply(_))
-  .catchSome {
-    bar
-  }
+foo >>= (_.http(registry, port).map(Option.apply(_)).catchSome {
+  bar
+})