From 62ac97d16a9231b71de84a50b3c0d32f554b34e4 Mon Sep 17 00:00:00 2001 From: Dmitry Petrashko Date: Wed, 15 Apr 2015 21:15:05 +0200 Subject: Refactor BackendInterface.Select: desuggar only once. --- .../tools/backend/jvm/DottyBackendInterface.scala | 30 ++++++++++++++-------- 1 file changed, 20 insertions(+), 10 deletions(-) (limited to 'src/dotty/tools/backend/jvm/DottyBackendInterface.scala') diff --git a/src/dotty/tools/backend/jvm/DottyBackendInterface.scala b/src/dotty/tools/backend/jvm/DottyBackendInterface.scala index bdba2935e..a1137896b 100644 --- a/src/dotty/tools/backend/jvm/DottyBackendInterface.scala +++ b/src/dotty/tools/backend/jvm/DottyBackendInterface.scala @@ -858,18 +858,28 @@ class DottyBackendInterface()(implicit ctx: Context) extends BackendInterface{ object Select extends SelectDeconstructor { - override def isEmpty: Boolean = field match { - case t: tpd.Select => false - case t: tpd.Ident => desugarIdent(t).isEmpty - case _ => true - } + var desugared: tpd.Select = null + + override def isEmpty: Boolean = + desugared eq null + + def _1: Tree = desugared.qualifier - def _1: Tree = field match { - case t: tpd.Select => t.qualifier - case t: tpd.Ident => desugarIdent(t).get + def _2: Name = desugared.name + + override def unapply(s: Select): this.type = { + s match { + case t: Select => desugared = t + case t: Ident => + desugarIdent(t) match { + case Some(t) => desugared = t + case None => desugared = null + } + case _ => desugared = null + } + + this } - def _2: Name = field.name - } object Apply extends ApplyDeconstructor { -- cgit v1.2.3