| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
The access boundary of a def in a local class is the owner of that
class, previously it was set to the access boundary of the owner of the
class instead.
|
|
|
|
|
|
|
|
|
|
|
| |
Previously Outer2#Inner#foo failed to compile with:
```
non-private method foo refers to private value x in its type signature
```
This should compile because the boundary of `foo` is `class Outer2`
and the boundary of `x` is `object Outer2`. This commit fixes this by
also considering the linked boundary in `checkNoPrivateLeaks`.
|
|
|
|
|
|
|
|
|
|
| |
Previously, the tests where all in tests/pos/java-interop which means
that partest would try to run them as one single test, this failed
because some of these tests define classes with the same name. We could
fix this by putting them all in separate packages but for now it's
simple to move them to tests/pos-java-interop (they will still be run
individually as they should thanks to the `java_all` test defined in
tests.scala)
|
|
|
|
|
|
|
|
| |
We forgot the case where a hoistable method can still
refer to free variables that have to be passed using
outer pointers.
Fixes #1664.
|
|\
| |
| | |
Fix #1638: Don't import when typing a package clause
|
| |
| |
| |
| |
| |
| |
| | |
When typing the first identifier of a package clause, disable
imports, as package clauses are never imported.
Fixes #1638.
|
|/
|
|
|
|
|
| |
Drop special mode that handles future defs without which we
get DenotationNotDefinedHere errors. In more than a year, this
has only turned up false negatives. So I think it's better to
drop the check, and the contortions needed to deal with it.
|
|\
| |
| | |
Fix-#1500 Include constraining type variables when interpolating
|
| |
| |
| |
| | |
Fixes #1500. Review by @smarter.
|
|\ \
| | |
| | | |
Fix #1570: Allow inline parameters as inline args
|
| |/
| |
| |
| |
| |
| |
| | |
Inline parameters can always be passed to other
inline parameters.
Fixes #1570.
|
| | |
|
|\ \
| | |
| | | |
Fix #1576: Parameters of Java annotations should be public
|
| |/
| |
| |
| |
| | |
Parameters of annotation classes parsed by the Java parser should
have public val parameters. Otherwise they cannot be accessed as fields.
|
|\ \
| | |
| | | |
Fix #1567: Widen private constructor in value class
|
| |/
| |
| |
| |
| |
| |
| |
| | |
Private or protected constructors of value classes need to be widenened
to public in order to enable boxing anywhere.
Technically we should also do something about qualified private constructors, but since we
want to get rid of them anyway it's urgent.
|
|\ \
| | |
| | | |
Fix #1590: Eliminate wildcards when approximating a type
|
| | | |
|
| |/
| |
| |
| |
| | |
Fixes #1590. Type variables should never be instantiated to types
containing wildcards.
|
|\ \
| | |
| | | |
Fix #1544: Allow long signatures in names
|
| | |
| | |
| | |
| | |
| | | |
Fixes #1544 by making the length field use 1 or 2 bytes,
depending on the number of parameters in a signature.
|
|\ \ \
| | | |
| | | | |
Fix #1540: overloaded get and isDefined in option-less patmat
|
| | | | |
|
| | |/
| |/| |
|
|/ /
| |
| |
| | |
Fixes #1515. Review by @smarter.
|
| | |
|
| |
| |
| |
| |
| |
| | |
Let them inherit the same traits and push as much
functionality as possibly into the common superclass
GenericType.
|
| | |
|
| | |
|
|/ |
|
|\
| |
| | |
Implement inline
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
`inline` is now a modifier keyword. To keep disruption tolerable,
we still allow `@inline` as an annotation as well. Other uses of
`inline` are supported only under `-language:Scala2` and are rewritten
to identifiers in backticks.
|
|\ \
| |/
|/| |
[WIP] fix encoding issues
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
rename test/pos/valueclasses to pos_valueclasses
tests/pos/valueclasses generates a valueclasses.flags
file in /tests/partest-generated/pos that conflicts
with the valueClasses.flags file that
tests/neg/valueClasses.scala tries to create
|
|\ \
| | |
| | | |
Fix #1457: Three incompatbilities with scalac
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Two of these are unavoidable. I moved the tests to diabled/not-representable
and added in each case a comment to the main scala file detailing why there
is a deviation.
The last one (import-rewrite) is fixed.
|
|\ \ \
| | | |
| | | | |
Fix #1513: misaligned by name type parameter type bounds
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This commit removes a problematic duplicated `checkBounds` call on `TypeApply`.
To verify correctness of this change on has to check that `normalizeTree` used only once [1],
and the function using `normalizeTree` already takes care of calling `checkBounds`.
[1]: https://github.com/lampepfl/dotty/blob/0e8f05d88bfef95fac59f522fd9d06792126bd11/src/dotty/tools/dotc/transform/PostTyper.scala#L205
|
| | | | |
|
| | | | |
|
|/ / / |
|
|\ \ \
| | | |
| | | | |
Fix #1335: Generate null checks for extractors
|
| | |/
| |/| |
|
|/ /
| |
| |
| |
| |
| | |
The issue fixed here was introduced by 71027f15. The added `csyms.isEmpty` condition on `case nil =>` is always true, which is clearely a bug.
t1500c still fails with covariant (or contravariant) type parameters on `::`, but this seams to be a more complicated issue involving the typer.
|
| |
| |
| |
| |
| |
| | |
type.
Test case is isApplicableSafe -Ycheck:first.
|
| | |
|
| |
| |
| |
| | |
We did not properly rename parameter references before.
|
| |
| |
| |
| |
| |
| | |
We now always widen selector type to the superclass if necessary,
no matter whether the selector type refers to a trait or a proper
class.
|