AvoidInfix: don't duplicate RedundantParens logic

scalameta · Jun 1, 2022 · 52d21c3 · 52d21c3
1 parent 37e606c
commit 52d21c3
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 16 deletions.
diff --git a/scalafmt-core/shared/src/main/scala/org/scalafmt/rewrite/AvoidInfix.scala b/scalafmt-core/shared/src/main/scala/org/scalafmt/rewrite/AvoidInfix.scala
@@ -93,20 +93,6 @@ class AvoidInfix(implicit ctx: RewriteCtx) extends RewriteSession {
           builder += TokenPatch.AddRight(lhsLast, ")", keepTok = true)
         }
 
-        // remove parens if enclosed
-        for {
-          parent <- tree.parent
-          if !parent.is[Term.ApplyInfix] && !parent.is[Term.Apply]
-          if ctx.style.rewrite.rules.contains(RedundantParens)
-          infixTokens = tree.tokens
-          head <- infixTokens.headOption if head.is[Token.LeftParen]
-          last <- infixTokens.lastOption if last.is[Token.RightParen]
-          if ctx.isMatching(head, last)
-        } yield {
-          builder += TokenPatch.Remove(head)
-          builder += TokenPatch.Remove(last)
-        }
-
         ctx.addPatchSet(builder.result(): _*)
 
       case _ =>

diff --git a/scalafmt-tests/src/test/resources/rewrite/AvoidInfix.stat b/scalafmt-tests/src/test/resources/rewrite/AvoidInfix.stat
@@ -422,7 +422,5 @@ object a {
 >>>
 object a {
   val foo = bar
-
   a.baz(b)
-
 }