From ef2fe94a9ced693444021b64bd95666ed2a22990 Mon Sep 17 00:00:00 2001 From: Albert Meltzer <7529386+kitbellew@users.noreply.github.com> Date: Sat, 11 Feb 2023 19:00:09 -0800 Subject: [PATCH] Scalameta: don't use deprecated methods (v4.7.4) --- .../scala/org/scalafmt/internal/FormatOps.scala | 3 +-- .../main/scala/org/scalafmt/util/TreeOps.scala | 16 ++++------------ 2 files changed, 5 insertions(+), 14 deletions(-) diff --git a/scalafmt-core/shared/src/main/scala/org/scalafmt/internal/FormatOps.scala b/scalafmt-core/shared/src/main/scala/org/scalafmt/internal/FormatOps.scala index 798ad77ecc..4d3f89ece7 100644 --- a/scalafmt-core/shared/src/main/scala/org/scalafmt/internal/FormatOps.scala +++ b/scalafmt-core/shared/src/main/scala/org/scalafmt/internal/FormatOps.scala @@ -471,7 +471,7 @@ class FormatOps( val fullInfix = InfixSplits.findEnclosingInfix(app) val ok = isEnclosedInParens(fullInfix) || fullInfix.parent.forall { case t: Defn.Val => t.rhs eq fullInfix - case t: Defn.Var => t.rhs.contains(fullInfix) + case t: Defn.Var => t.body eq fullInfix case _ => true } if (ok) @@ -2650,7 +2650,6 @@ class FormatOps( def getBlocks(ft: FormatToken, nft: FormatToken, all: Boolean): Result = ft.meta.leftOwner match { case t: Ctor.Secondary => Some((t, seq(all, t.init, t.stats))) - case t: Defn.Var => t.rhs.map(_ -> Nil) case t: Tree.WithBody => Some((t.body, Nil)) case _ => BlockImpl.getBlocks(ft, nft, all) } diff --git a/scalafmt-core/shared/src/main/scala/org/scalafmt/util/TreeOps.scala b/scalafmt-core/shared/src/main/scala/org/scalafmt/util/TreeOps.scala index fce249909b..09878a157c 100644 --- a/scalafmt-core/shared/src/main/scala/org/scalafmt/util/TreeOps.scala +++ b/scalafmt-core/shared/src/main/scala/org/scalafmt/util/TreeOps.scala @@ -332,16 +332,8 @@ object TreeOps { @tailrec def defDefBody(tree: Tree): Option[Tree] = tree match { - case d: Defn.Def => Some(d.body) - case d: Defn.Macro => Some(d.body) - case d: Defn.Given => Some(d.templ) - case d: Defn.GivenAlias => Some(d.body) - case d: Defn.Val => Some(d.rhs) - case d: Defn.Var => d.rhs - case t: Defn.Class => Some(t.templ) - case t: Defn.Trait => Some(t.templ) - case t: Defn.Enum => Some(t.templ) - case t: Defn.ExtensionGroup => Some(t.body) + case d: Defn with Tree.WithBody => Some(d.body) + case d: Defn with Stat.WithTemplate => Some(d.templ) case t: Ctor.Secondary => Some(t.init) case _: Ctor.Primary | _: Pat.Var | _: Term.Name => tree.parent match { @@ -451,7 +443,6 @@ object TreeOps { new FormatToken.ExtractFromMeta(_.leftOwner match { case _: Enumerator => None // it's WithBody case t: Ctor.Secondary => Some(t.init) - case t: Defn.Var => t.rhs.orElse(Some(t)) // `_` replaced with None case t: Tree.WithBody => Some(t.body) case t: Term.Param => t.default case _ => None @@ -739,7 +730,8 @@ object TreeOps { t.parent match { case Some(ts: Term.Select) if ts.name.value == "stripMargin" => ts.parent match { - case Some(Term.Apply(_, List(arg: Lit.Char))) => Some(arg.value) + case Some(Term.Apply.Initial(_, List(arg: Lit.Char))) => + Some(arg.value) case _ => Some('|') } case _ => None