Remove unused code (#2900)

zio · Oct 2, 2023 · f9c68d8 · f9c68d8
1 parent 4ef60ee
commit f9c68d8
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/quill-engine/src/main/scala/io/getquill/ast/StatelessTransformer.scala b/quill-engine/src/main/scala/io/getquill/ast/StatelessTransformer.scala
@@ -2,7 +2,7 @@ package io.getquill.ast
 
 trait StatelessTransformer {
 
-  def applyIdent(id: Ident) = id
+  def applyIdent(id: Ident): Ident = id
 
   def apply(e: Ast): Ast =
     e match {

diff --git a/quill-engine/src/main/scala/io/getquill/norm/capture/DemarcateExternalAliases.scala b/quill-engine/src/main/scala/io/getquill/norm/capture/DemarcateExternalAliases.scala
@@ -20,10 +20,10 @@ import io.getquill.ast._
  * body. Other places where this needs to be done (e.g. in a Tuple that
  * `Returning` returns) are done in `ExpandReturning`.
  */
-private[getquill] case class DemarcateExternalAliases(externalIdent: Ident) extends StatelessTransformer {
+private[getquill] final case class DemarcateExternalAliases(externalIdent: Ident) extends StatelessTransformer {
 
-  def applyNonOverride(idents: Ident*)(ast: Ast) =
-    if (idents.forall(_ != externalIdent)) apply(ast)
+  private def applyNonOverride(idents: Ident*)(ast: Ast): Ast =
+    if (!idents.contains(externalIdent)) apply(ast)
     else ast
 
   override def apply(ast: Ast): Ast = ast match {
@@ -71,7 +71,7 @@ private[getquill] case class DemarcateExternalAliases(externalIdent: Ident) exte
 
 object DemarcateExternalAliases {
 
-  private def demarcateQueriesInBody(id: Ident, body: Ast) =
+  private def demarcateQueriesInBody(id: Ident, body: Ast): Ast =
     Transform(body) {
       // Apply to the AST defined apply method about, not to the superclass method that takes Query
       case q: Query => new DemarcateExternalAliases(id).apply(q.asInstanceOf[Ast])
@@ -81,7 +81,6 @@ object DemarcateExternalAliases {
     case Returning(a, id, body) =>
       Returning(a, id, demarcateQueriesInBody(id, body))
     case ReturningGenerated(a, id, body) =>
-      val d = demarcateQueriesInBody(id, body)
       ReturningGenerated(a, id, demarcateQueriesInBody(id, body))
     case other =>
       other