diff options
| author | Martin Odersky <odersky@gmail.com> | 2014-11-14 16:11:31 +0100 |
|---|---|---|
| committer | Martin Odersky <odersky@gmail.com> | 2014-11-14 16:11:31 +0100 |
| commit | 4d5a901d10a28c286f8754134f5030daae0d239b (patch) | |
| tree | 41936bfc93fb242db330aeac3627038e9c1ef142 /src/dotty/tools/dotc/typer/RefChecks.scala | |
| parent | 9dae49b8994f72f2b4b83665c53a63b49896c5de (diff) | |
| download | dotty-4d5a901d10a28c286f8754134f5030daae0d239b.tar.gz dotty-4d5a901d10a28c286f8754134f5030daae0d239b.tar.bz2 dotty-4d5a901d10a28c286f8754134f5030daae0d239b.zip | |
Fix OverridingPairs
OverridingPairs had several bugs which caused pairs to be lost, which caused
missing overrides checks. Fixing OverridingPairs revealed several test failures
(and a problem in Synthetics generation which was fixed in the last commit).
Tests that became negative are all moved into neg/overrides.scala, and the
original versions in pos were fixed.
Diffstat (limited to 'src/dotty/tools/dotc/typer/RefChecks.scala')
| -rw-r--r-- | src/dotty/tools/dotc/typer/RefChecks.scala | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/dotty/tools/dotc/typer/RefChecks.scala b/src/dotty/tools/dotc/typer/RefChecks.scala index f45966741..2ae9a5b97 100644 --- a/src/dotty/tools/dotc/typer/RefChecks.scala +++ b/src/dotty/tools/dotc/typer/RefChecks.scala @@ -246,7 +246,7 @@ object RefChecks { } else if (!other.is(Deferred) && member.isClass) { overrideError("cannot be used here - classes can only override abstract types") } else if (other.isEffectivelyFinal) { // (1.2) - overrideError("cannot override final member") + overrideError(i"cannot override final member ${other.showLocated}") } else if (!other.is(Deferred) && !isDefaultGetter(other.name) && !member.is(AnyOverrideOrSynthetic)) { // (*) Synthetic exclusion for (at least) default getters, fixes SI-5178. We cannot assign the OVERRIDE flag to // the default getter: one default getter might sometimes override, sometimes not. Example in comment on ticket. |