Skip to content

Commit 42ad90f

Browse files
committed
prevPhase should indicate previous phase in explicitOuter
Makes it less confusing as thisTransformer is already used for the ExplicitOuter itself
1 parent e330cc3 commit 42ad90f

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/dotty/tools/dotc/transform/ExplicitOuter.scala

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -111,11 +111,11 @@ object ExplicitOuter {
111111
/** Ensure that class `cls` has outer accessors */
112112
def ensureOuterAccessors(cls: ClassSymbol)(implicit ctx: Context): Unit = {
113113
//todo: implementing #165 would simplify this logic
114-
val thisTransformer = ctx.phase.prev
115-
assert(thisTransformer.id <= ctx.explicitOuter.id, "can add $outer symbols only before ExplicitOuter")
116-
assert(thisTransformer.isInstanceOf[DenotTransformer], "adding outerAccessors requires being DenotTransformer")
114+
val prevPhase = ctx.phase.prev
115+
assert(prevPhase.id <= ctx.explicitOuter.id, "can add $outer symbols only before ExplicitOuter")
116+
assert(prevPhase.isInstanceOf[DenotTransformer], "adding outerAccessors requires being DenotTransformer")
117117
if (!hasOuter(cls)) {
118-
newOuterAccessors(cls).foreach(_.enteredAfter(thisTransformer.asInstanceOf[DenotTransformer]))
118+
newOuterAccessors(cls).foreach(_.enteredAfter(prevPhase.asInstanceOf[DenotTransformer]))
119119
}
120120
}
121121

0 commit comments

Comments
 (0)