First steps towards rewriting from Scala2 in dotty #1154
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| vdef1 | ||
| } | ||
|
|
||
| /** Add a @volitile to lazy vals when rewriting from Scala2 */ |
Firs version of patching that can be invoked by dotty compiler itself.
Driver should not know that patch functionality exists. Instead, introduce settings that can introduce their own stateful values.
No more leaking ofMove PatchedFiles in a settings option. Move all patch classes into a `Rewrites` object.
Map typed to corresponding untyped trees.
Imports are missing afterwards.
Selectors should be defs, not lazy vals.
Some random neg tests from previous experiments.
Revert this commit once scala#1149 is fixed.
Scala2 allows `x _` even if `x` is not a method. Dotty disallows them. The patch removes the ` _` in these cases.
Test rewritings that were implemented so far.
Remove println; add docs
Gave overlapping positions in the case of longer lists of children.
`Iterator.sliding(2, 1)` returns a one-element result if the original iterator contains only one element, which makes it unpleasant to use for our task. Replaced by a fold.
A constructor
def this() { ... }
needs to be rewritten to
def this() = { ... }
not to
def this(): Unit = { ... }
1. trailing `_`:
`x _` is rewritten to `(() => x)` not to `x`
2. lazy vals:
Rewrites are done in Typer, not LazyVals. Later on we are too much at risk to
hit synthetically generated lazy vals.
|
Any objections to merge this one? |
|
I'll review it today. |
| if (ctx.scala2Mode && sym.owner.isConstructor) | ||
| ctx.migrationWarning(s"According to new variance rules, this is no longer accepted; need to annotate with @uncheckedVariance:\n$msg", sym.pos) | ||
| if (ctx.scala2Mode && sym.owner.isConstructor) { | ||
| ctx.migrationWarning(s"According to new variance rules, this is no longer accepted; need to annotate with @uncheckedVariance:\n$msg, pos = ${sym.pos}", sym.pos) |
There was a problem hiding this comment.
I don't think we need pos = ${sym.pos} in the migration warning
| * | ||
| * Otherwise, in case there is exactly one variable x_1 in pattern | ||
| * val/var p = e ==> val/var x_1 = (e: @unchecked) match (case p => (x_1)) | ||
| * val/var/lazy val p = e ==> val/var x_1 = (e: @unchecked) match (case p => (x_1)) |
There was a problem hiding this comment.
val/var x_1 should be val/var/lazy val x_1 I think
|
Probably not worth fixing if it's complicated to deal with but note that: object Foo {
lazy val x, y = 1
}gets rewritten to: object Foo {
@volatile @volatile lazy val x, y = 1
} |
odersky
added a commit
that referenced
this pull request
Mar 18, 2016
First steps towards rewriting from Scala2 in dotty
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is an attempt to do the rewriting right in the dotty compiler. The logic is, if dotty can detect
Scala2 incompatibilities it should be able to apply fixes at the same time. That way there is no error-prone syncing between different tools.
This PR
There is also a change the way annotations are propagated to derived symbols.
With this PR, stdlib can be patched and recompiled without -language:Scala2 (pending fix of
@volatile implicit issues).