-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Rework NamedTypes #3494
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Rework NamedTypes #3494
Changes from 57 commits
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
94f8041
Generalize symbolic named types
odersky 9d962b6
Change isLocallyDefined criterion for TreePickler
odersky 30395c8
Cater for external symbolic references in pickler
odersky e0866d4
Allow pickling of general symbolic references
odersky 68760a4
Create OverloadedSymbol and OverloadedDenotation
odersky 0864141
More robust implementation of requiredMethod
odersky 12c4429
Optionally make NamedTypes symbolic
odersky 2399c68
Fix Erasure's transformDenot for SingleDenotations
odersky cea1823
Drop OverloadedSymbol/Denotation
odersky 8326890
Don't interpolate or simplify overloaded references
odersky a9291b4
Rework denotations
odersky 7e32751
Handle shadowed refs under new scheme
odersky d3b04aa
Refine Erasure#transform
odersky c247850
Fixes to handling of overloaded references
odersky 7069e63
Bugfix in ensureAccessible
odersky 17606c9
Better diagnostics for implicit shadowing errors
odersky 0a471ee
Improve "not found" message.
odersky 5b6657d
Better error message for merge errors
odersky fc85a41
Never change the symbol of a symbolic NamedType
odersky 5e584a8
Fixes to the new scheme
odersky d42da02
Commit to new scheme where symbols of symbolic references never change
odersky 6aa350a
Drop hack that hides stale package members
odersky 3d63680
Fix definition of Throwable
odersky a76e884
Refine determination when denotations can be private
odersky f1d0dbe
Avoid stackoverflow in compileNeg
odersky b363633
Remove special case in substSym
odersky 89ec7d6
Cleanup
odersky 6b6523a
Replace references to hasFixedSym
odersky c4f0a3e
Weaken post condition in FirstTransform
odersky da99504
Always reload denotation with initial name of last symbol.
odersky 614daaa
Avoid stale symbol errors via isStatic
odersky 1ad2157
Refine bringForward
odersky 18a16e9
Fix pickling of refs to shadowed inner classes
odersky 03d7e3f
Allow -uniqid printing on the fly.
odersky 5b818c4
Revert implementation of stableInRunSymbol
odersky 4dea7de
Allow for aut-updating of package members
odersky 37a76d3
Make isStatic force less.
odersky 65b30a2
Add missing case in TypeComparer
odersky 25af9f6
Switch to new Scheme
odersky 6dcfc63
Move test to pending
odersky 49b78c3
Fail on all stale references by default
odersky c3be42e
Change treatment of signatures in TermRefs
odersky 24ca6c0
Drop all code under !newScheme
odersky a1ae7f9
Drop NamedType.withSym
odersky 939d1e2
Replace invalidateDenot by reloadDenot
odersky 5b49561
Change visibility of some NamedType operations
odersky 0c87b7b
Resolve previously marked issues
odersky c3f43f9
Polishings
odersky 6953dfb
Make sure bringForward returns a denotation valid in current period
odersky 68cd5e6
Handle import aliases of implicit definitions
odersky f33529c
Fix rebase breakage
odersky c81f282
Rename ImplicitDef -> ImplicitRef
odersky 6ff60da
More tests
odersky 8578fa1
Add original test that caused the whole rewrite
odersky dd95817
Fix rebase breakage
odersky eb93112
Harden IDE
odersky 2524788
Some renamings
odersky 759af1e
Adress reviewers comments
odersky File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we still need this? It's currently never enabled.