Skip to content

Commit e3e66ef

Browse files
committed
Rename SimpleTermName -> SimpleName, DerivedTermName -> DerivedName
1 parent 2f7c678 commit e3e66ef

File tree

10 files changed

+101
-98
lines changed

10 files changed

+101
-98
lines changed

compiler/src/dotty/tools/dotc/core/Denotations.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1195,7 +1195,7 @@ object Denotations {
11951195
select(recur(prefix), wrap(selector))
11961196
case qn @ AnyQualifiedName(prefix, _) =>
11971197
recur(prefix, n => wrap(qn.info.mkString(n).toTermName))
1198-
case path: SimpleTermName =>
1198+
case path: SimpleName =>
11991199
def recurSimple(len: Int, wrap: TermName => Name): Denotation = {
12001200
val point = path.lastIndexOf('.', len - 1)
12011201
val selector = wrap(path.slice(point + 1, len).asTermName)

compiler/src/dotty/tools/dotc/core/NameKinds.scala

Lines changed: 31 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -11,18 +11,21 @@ import Decorators._
1111
import Contexts.Context
1212
import collection.mutable
1313

14+
/** Defines possible kinds of NameInfo of a derived name */
1415
object NameKinds {
1516

1617
@sharable private val simpleNameKinds = new mutable.HashMap[Int, ClassifiedNameKind]
1718
@sharable private val qualifiedNameKinds = new mutable.HashMap[Int, QualifiedNameKind]
1819
@sharable private val uniqueNameKinds = new mutable.HashMap[String, UniqueNameKind]
1920

21+
/** A class for the info stored in a derived name */
2022
abstract class NameInfo extends DotClass {
2123
def kind: NameKind
2224
def mkString(underlying: TermName): String
23-
def map(f: SimpleTermName => SimpleTermName): NameInfo = this
25+
def map(f: SimpleName => SimpleName): NameInfo = this
2426
}
2527

28+
/** The kind of a derived name info */
2629
abstract class NameKind(val tag: Int) extends DotClass { self =>
2730
type ThisInfo <: Info
2831
class Info extends NameInfo { this: ThisInfo =>
@@ -31,12 +34,12 @@ object NameKinds {
3134
override def toString = infoString
3235
}
3336
def definesNewName = false
34-
def unmangle(name: SimpleTermName): TermName = name
37+
def unmangle(name: SimpleName): TermName = name
3538
def mkString(underlying: TermName, info: ThisInfo): String
3639
def infoString: String
3740
}
3841

39-
object SimpleTermNameKind extends NameKind(UTF8) { self =>
42+
object SimpleNameKind extends NameKind(UTF8) { self =>
4043
type ThisInfo = Info
4144
val info = new Info
4245
def mkString(underlying: TermName, info: ThisInfo) = unsupported("mkString")
@@ -48,8 +51,8 @@ object NameKinds {
4851
val info = new Info
4952
def apply(qual: TermName) =
5053
qual.derived(info)
51-
def unapply(name: DerivedTermName): Option[TermName] = name match {
52-
case DerivedTermName(underlying, `info`) => Some(underlying)
54+
def unapply(name: DerivedName): Option[TermName] = name match {
55+
case DerivedName(underlying, `info`) => Some(underlying)
5356
case _ => None
5457
}
5558
simpleNameKinds(tag) = this
@@ -59,44 +62,44 @@ object NameKinds {
5962
extends ClassifiedNameKind(tag, if (optInfoString.isEmpty) s"Prefix $prefix" else optInfoString) {
6063
def mkString(underlying: TermName, info: ThisInfo) =
6164
underlying.mapLast(n => termName(prefix + n.toString)).toString
62-
override def unmangle(name: SimpleTermName): TermName =
65+
override def unmangle(name: SimpleName): TermName =
6366
if (name.startsWith(prefix)) apply(name.drop(prefix.length).asSimpleName)
6467
else name
6568
}
6669

6770
class SuffixNameKind(tag: Int, suffix: String, optInfoString: String = "")
6871
extends ClassifiedNameKind(tag, if (optInfoString.isEmpty) s"Suffix $suffix" else optInfoString) {
6972
def mkString(underlying: TermName, info: ThisInfo) = underlying.toString ++ suffix
70-
override def unmangle(name: SimpleTermName): TermName =
73+
override def unmangle(name: SimpleName): TermName =
7174
if (name.endsWith(suffix)) apply(name.take(name.length - suffix.length).asSimpleName)
7275
else name
7376
}
7477

7578
trait QualifiedInfo extends NameInfo {
76-
val name: SimpleTermName
79+
val name: SimpleName
7780
}
7881

7982
class QualifiedNameKind(tag: Int, val separator: String)
8083
extends NameKind(tag) {
8184
type ThisInfo = QualInfo
82-
case class QualInfo(val name: SimpleTermName) extends Info with QualifiedInfo {
83-
override def map(f: SimpleTermName => SimpleTermName): NameInfo = new QualInfo(f(name))
85+
case class QualInfo(val name: SimpleName) extends Info with QualifiedInfo {
86+
override def map(f: SimpleName => SimpleName): NameInfo = new QualInfo(f(name))
8487
override def toString = s"$infoString $name"
8588
}
86-
def apply(qual: TermName, name: SimpleTermName): TermName =
89+
def apply(qual: TermName, name: SimpleName): TermName =
8790
qual.derived(new QualInfo(name))
8891

8992
/** Overloaded version used only for ExpandedName and TraitSetterName.
9093
* Needed because the suffix of an expanded name may itself be expanded.
9194
* For example, look at javap of scala.App.initCode
9295
*/
9396
def apply(qual: TermName, name: TermName): TermName = name rewrite {
94-
case name: SimpleTermName => apply(qual, name)
97+
case name: SimpleName => apply(qual, name)
9598
case AnyQualifiedName(_, _) => apply(qual, name.toSimpleName)
9699
}
97100

98-
def unapply(name: DerivedTermName): Option[(TermName, SimpleTermName)] = name match {
99-
case DerivedTermName(qual, info: this.QualInfo) => Some((qual, info.name))
101+
def unapply(name: DerivedName): Option[(TermName, SimpleName)] = name match {
102+
case DerivedName(qual, info: this.QualInfo) => Some((qual, info.name))
100103
case _ => None
101104
}
102105

@@ -110,8 +113,8 @@ object NameKinds {
110113
}
111114

112115
object AnyQualifiedName {
113-
def unapply(name: DerivedTermName): Option[(TermName, SimpleTermName)] = name match {
114-
case DerivedTermName(qual, info: QualifiedInfo) =>
116+
def unapply(name: DerivedName): Option[(TermName, SimpleName)] = name match {
117+
case DerivedName(qual, info: QualifiedInfo) =>
115118
Some((name.underlying, info.name))
116119
case _ => None
117120
}
@@ -128,11 +131,11 @@ object NameKinds {
128131
}
129132
def apply(qual: TermName, num: Int) =
130133
qual.derived(new NumberedInfo(num))
131-
def unapply(name: DerivedTermName): Option[(TermName, Int)] = name match {
132-
case DerivedTermName(underlying, info: this.NumberedInfo) => Some((underlying, info.num))
134+
def unapply(name: DerivedName): Option[(TermName, Int)] = name match {
135+
case DerivedName(underlying, info: this.NumberedInfo) => Some((underlying, info.num))
133136
case _ => None
134137
}
135-
protected def skipSeparatorAndNum(name: SimpleTermName, separator: String): Int = {
138+
protected def skipSeparatorAndNum(name: SimpleName, separator: String): Int = {
136139
var i = name.length
137140
while (i > 0 && name(i - 1).isDigit) i -= 1
138141
if (i > separator.length && i < name.length &&
@@ -142,8 +145,8 @@ object NameKinds {
142145
}
143146

144147
object AnyNumberedName {
145-
def unapply(name: DerivedTermName): Option[(TermName, Int)] = name match {
146-
case DerivedTermName(qual, info: NumberedInfo) => Some((qual, info.num))
148+
def unapply(name: DerivedName): Option[(TermName, Int)] = name match {
149+
case DerivedName(qual, info: NumberedInfo) => Some((qual, info.num))
147150
case _ => None
148151
}
149152
}
@@ -163,8 +166,8 @@ object NameKinds {
163166
}
164167

165168
object AnyUniqueName {
166-
def unapply(name: DerivedTermName): Option[(TermName, String, Int)] = name match {
167-
case DerivedTermName(qual, info: NumberedInfo) =>
169+
def unapply(name: DerivedName): Option[(TermName, String, Int)] = name match {
170+
case DerivedName(qual, info: NumberedInfo) =>
168171
info.kind match {
169172
case unique: UniqueNameKind => Some((qual, unique.separator, info.num))
170173
case _ => None
@@ -181,7 +184,7 @@ object NameKinds {
181184
private val FalseSuper = termName("$$super")
182185
private val FalseSuperLength = FalseSuper.length
183186

184-
override def unmangle(name: SimpleTermName): TermName = {
187+
override def unmangle(name: SimpleName): TermName = {
185188
var i = name.lastIndexOfSlice(str.EXPAND_SEPARATOR)
186189
if (i < 0) name
187190
else {
@@ -220,7 +223,7 @@ object NameKinds {
220223
val LiftedTreeName = new UniqueNameKind("liftedTree")
221224

222225
val UniqueExtMethName = new UniqueNameKind("$extension") {
223-
override def unmangle(name: SimpleTermName): TermName = {
226+
override def unmangle(name: SimpleName): TermName = {
224227
val i = skipSeparatorAndNum(name, separator)
225228
if (i > 0) {
226229
val index = name.drop(i).toString.toInt
@@ -245,7 +248,7 @@ object NameKinds {
245248
prefix.toString + str.DEFAULT_GETTER + (info.num + 1)
246249
}
247250
// TODO: Reduce code duplication with UniqueExtMethName
248-
override def unmangle(name: SimpleTermName): TermName = {
251+
override def unmangle(name: SimpleName): TermName = {
249252
val i = skipSeparatorAndNum(name, str.DEFAULT_GETTER)
250253
if (i > 0) {
251254
val index = name.drop(i).toString.toInt - 1
@@ -296,8 +299,8 @@ object NameKinds {
296299

297300
def apply(qual: TermName, sig: Signature) =
298301
qual.derived(new SignedInfo(sig))
299-
def unapply(name: DerivedTermName): Option[(TermName, Signature)] = name match {
300-
case DerivedTermName(underlying, info: SignedInfo) => Some((underlying, info.sig))
302+
def unapply(name: DerivedName): Option[(TermName, Signature)] = name match {
303+
case DerivedName(underlying, info: SignedInfo) => Some((underlying, info.sig))
301304
case _ => None
302305
}
303306

compiler/src/dotty/tools/dotc/core/NameOps.scala

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,8 @@ object NameOps {
5252
implicit class NameDecorator[N <: Name](val name: N) extends AnyVal {
5353
import nme._
5454

55-
def testSimple(f: SimpleTermName => Boolean): Boolean = name match {
56-
case name: SimpleTermName => f(name)
55+
def testSimple(f: SimpleName => Boolean): Boolean = name match {
56+
case name: SimpleName => f(name)
5757
case name: TypeName => name.toTermName.testSimple(f)
5858
case _ => false
5959
}
@@ -83,7 +83,7 @@ object NameOps {
8383
def isOpAssignmentName: Boolean = name match {
8484
case raw.NE | raw.LE | raw.GE | EMPTY =>
8585
false
86-
case name: SimpleTermName =>
86+
case name: SimpleName =>
8787
name.length > 0 && name.last == '=' && name.head != '=' && isOperatorPart(name.head)
8888
case _ =>
8989
false
@@ -101,7 +101,7 @@ object NameOps {
101101
*/
102102
def stripModuleClassSuffix: N = likeSpaced {
103103
val semName = name.toTermName match {
104-
case name: SimpleTermName if name.endsWith("$") => name.unmangleClassName
104+
case name: SimpleName if name.endsWith("$") => name.unmangleClassName
105105
case _ => name
106106
}
107107
semName.exclude(ModuleClassName)
@@ -229,19 +229,19 @@ object NameOps {
229229
def compactified(implicit ctx: Context): TermName = termName(compactify(name.toString))
230230

231231
def unmangleClassName: N = name.toTermName match {
232-
case name: SimpleTermName
232+
case name: SimpleName
233233
if name.endsWith(str.MODULE_SUFFIX) && !nme.falseModuleClassNames.contains(name) =>
234234
likeSpaced(name.dropRight(str.MODULE_SUFFIX.length).moduleClassName)
235235
case _ => name
236236
}
237237

238238
def unmangle(kind: NameKind): N = likeSpaced {
239239
name rewrite {
240-
case unmangled: SimpleTermName =>
240+
case unmangled: SimpleName =>
241241
kind.unmangle(unmangled)
242242
case ExpandedName(prefix, last) =>
243243
kind.unmangle(last) rewrite {
244-
case kernel: SimpleTermName =>
244+
case kernel: SimpleName =>
245245
ExpandedName(prefix, kernel)
246246
}
247247
}

0 commit comments

Comments
 (0)