diff options
author | Adriaan Moors <adriaan.moors@typesafe.com> | 2015-09-01 17:06:33 -0700 |
---|---|---|
committer | Adriaan Moors <adriaan.moors@typesafe.com> | 2015-09-02 12:05:00 -0700 |
commit | df61ab67d3c32e8e996874206299938f5bd1584d (patch) | |
tree | 9b6e58c7cba8c78f744823a75930db71cf1d40f0 /src | |
parent | cc70cfb5024729935b5dd9be385847bd43c90b04 (diff) | |
download | scala-df61ab67d3c32e8e996874206299938f5bd1584d.tar.gz scala-df61ab67d3c32e8e996874206299938f5bd1584d.tar.bz2 scala-df61ab67d3c32e8e996874206299938f5bd1584d.zip |
Review feedback from retronym
Diffstat (limited to 'src')
-rw-r--r-- | src/compiler/scala/tools/nsc/typechecker/MethodSynthesis.scala | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/compiler/scala/tools/nsc/typechecker/MethodSynthesis.scala b/src/compiler/scala/tools/nsc/typechecker/MethodSynthesis.scala index 5fdddf0641..f3632b144d 100644 --- a/src/compiler/scala/tools/nsc/typechecker/MethodSynthesis.scala +++ b/src/compiler/scala/tools/nsc/typechecker/MethodSynthesis.scala @@ -338,10 +338,11 @@ trait MethodSynthesis { case _ => NoSymbol } - // TODO: when is `derivedSym.isOverloaded`??? is it always an error? - private def setterRhs = + private def setterRhs = { + assert(!derivedSym.isOverloaded, s"Unexpected overloaded setter $derivedSym for $basisSym in $enclClass") if (Field.noFieldFor(tree) || derivedSym.isOverloaded) EmptyTree else Assign(fieldSelection, Ident(setterParam)) + } private def setterDef = DefDef(derivedSym, setterRhs) override def derivedTree: Tree = if (setterParam == NoSymbol) EmptyTree else setterDef |