| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|\
| |
| | |
SI-5760: Improve error message for package$Klass conflict with Klass
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Added a clarification to DoubleDefError for when the previous symbol
was in the package object but current symbol is not.
This was actually supposed to be an opportunity to hack partest to
run the two-step failing compilation, but somebody beat me to it and my
rebase failed.
The next hacking opportunity might be to add .pt script files! The
possibilities are endless.
|
|\ \
| | |
| | | |
Test case closes SI-3880.
|
| |/
| |
| |
| | |
virtpatmat ftw.
|
|\ \
| | |
| | | |
Pending test for SI-3899.
|
| |/
| |
| |
| |
| |
| | |
The super accessor for the Java varargs method impedes Uncurry's
efforts to convert repeated arguments to an Array. I'm not sure how
to fix that.
|
|\ \
| | |
| | | |
Fix @varargs forwarder generation in the presence of nested templates.
|
| | |
| | |
| | |
| | | |
Uncomment a line, as was the intent.
|
| |/
| |
| |
| |
| |
| | |
Makes `newMembers` a Map[Symbol, Buffer[Tree]] to ensure we add the forwarders to the right template.
Closes SI-5125.
|
|\ \
| |/
|/|
| | |
Fix SI-4717: lazy val declared inside an anonymous class inside a specialized
context no longer crashes Duplicators.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
A lazy val declared inside an anonymous class inside a specialized
context no longer crashes Duplicators.
Previously, a duplicated lazy val was assigned to the wrong owner
in Duplicators:
def x[B >: A]: Unit = new Bounds[B] {
lazy val it = ??? // def or val okay
}
Above, the `it` in `$anon` in `x$mcZ$sp` had its owner set to `x$mcZ$sp`
instead of `$anon`. This crashed the typer when it had to retype its
lazy accessor, because there was no `lazy var it` in `$anon$`.
Furthermore, the duplicated symbol wasn't being added to the list of
declarations of `$anon`.
Changes:
1) `invalidate` in Duplicators takes an additional parameter which is
the new owner of the new symbol that has to be duplicated. If this parameter
is set to `NoSymbol`, then the new owner is `context.owner`, as before.
2) the newly created lazy val symbol is being added to the list of
declarations of its new owner.
Removes debugging output from the previous commit.
Review by dragos.
@mention dragos
|
| | |
|
|\ \
| | |
| | | |
Fix SI-4812: handle trait method with Symbol as default for argument
|
| | |
| | |
| | |
| | | |
Remove default arguments from parameter ValDefs in UnCurry.
|
|\ \ \
| | | |
| | | | |
fix SI-5044: better error message on cyclic error and named/default args
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | | |
Fix SI-2488: allow named arg, in original position, before positionals
|
| | | | | |
|
|\ \ \ \ \
| |_|_|/ /
|/| | | | |
Fix SI-5801: error messages regression.
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Fixes SI-4461.
|
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
No review.
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | | |
Fix SI-5544
|
| | |_|/ / /
| |/| | | |
| | | | | |
| | | | | | |
Type-check annotations in a context with a localDummy owner
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | | |
SIP-14: clean ups and fixes by @viktorklang
|
| | | | | | | |
|
|\ \ \ \ \ \ \
| |_|_|/ / / /
|/| | | | | | |
Fix 4138
|
| | |/ / / /
| |/| | | | |
|
|\ \ \ \ \ \
| |/ / / / /
|/| | | | | |
Test case for SI-3844 - fixed by #virtpatmat @adriaanm
|
| |/ / / / |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Fix SI-4928
|
| | |_|/ /
| |/| | |
| | | | |
| | | | | |
better error message when a parameter is first defined positionally, then with a named argument.
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Added infrastructure to enable easy enrichment of GenTraversables.
|
| | |/ / /
| |/| | | |
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Since ae5ff662, resetAttrs duplicates trees, which doesn't preserve ApplyConstructor.
My attempt to modify TreeCopier to do so proved trickier than expected.
In any case, ApplyConstructor is not long for this world, and is only used in tree printing
to distinguish `new X` from regular Apply trees, so this should suffice pending full surgery.
|
|\ \ \ \
| | | | |
| | | | | |
Fix SI-5259
|
| | | | |
| | | | |
| | | | |
| | | | | |
Calling the type checker on an Ident tree instead of using gen.mkAttributedRef assigns a SingleType to the tree.
|
|\ \ \ \ \
| |_|/ / /
|/| | | | |
Fix for SI-5610
|
| |/ / / |
|
| | | | |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | | |
Culling accumulated unnecessary code.
|
|/ / /
| | |
| | |
| | | |
And undeprecated Positional.
|
| | | | |
| \ \ | |
| \ \ | |
| \ \ | |
| \ \ | |
| \ \ | |
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | | |
'refs/pull/549/head'; commit 'refs/pull/550/head'; commit 'refs/pull/551/head' into develop
|
| | | | |/ / |
|
| | | |/ / |
|
| | | | |
| | | | |
| | | | |
| | | | | |
make TreeInfo recognize constructor calls after named arguments transformation.
|
|/ / / / |
|
| | | | | |
| \ \ \ | |
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | | |
'refs/pull/546/head' into develop
|