| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|\ \ \ \ \
| | | | | |
| | | | | | |
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
|
| |/ / / / /
| | | | | |
| | | | | |
| | | | | | |
virtpatmat strikes again.
|
| |\| | | | |
|
| | | | | | |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Don't print trees under -Xprint:all if they're identical to the
tree printed at the previous phase. It only works for a single
compilation unit but that is a huge step forward for us debuggers.
For instance this file:
trait Foo { def f = 5 }
used to produce 332 lines of output and now produces 92, with
zero loss of information. It ends with:
[[syntax trees at end of cleanup]] // a.scala: tree is unchanged since mixin
[[syntax trees at end of icode]] // a.scala: tree is unchanged since mixin
[[syntax trees at end of inliner]] // a.scala: tree is unchanged since mixin
[[syntax trees at end of inlineExceptionHandlers]] // a.scala: tree is unchanged since mixin
[[syntax trees at end of closelim]] // a.scala: tree is unchanged since mixin
[[syntax trees at end of dce]] // a.scala: tree is unchanged since mixin
[[syntax trees at end of jvm]] // a.scala: tree is unchanged since mixin
|
| | | | | | | |
| | \ \ \ \ | |
| | \ \ \ \ | |
| | \ \ \ \ | |
| | \ \ \ \ | |
| | \ \ \ \ | |
| | \ \ \ \ | |
| | \ \ \ \ | |
| | \ \ \ \ | |
| | \ \ \ \ | |
| | \ \ \ \ | |
| | \ \ \ \ | |
| | \ \ \ \ | |
| | \ \ \ \ | |
| |\ \ \ \ \ \ \ \ \ \ \ \
| | |_|_|_|_|_|_|_|/ / / /
| |/| | | | | | | | | | /
| | | | |_|_|_|_|_|_|_|/
| | | |/| | | | | | | | |
'scalamacros/topic/macrocherrypick', 'lrytz/t5626', 'lrytz/t5009', 'retronym/ticket/5029', 'retronym/ticket/4025', 'retronym/topic/quieter-nsdhnao' and 'retronym/ticket/1133' into develop
|