Skip to content
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

check if package names will be encoded in desugar #14597

Merged
merged 1 commit into from Mar 7, 2022

Conversation

bishabosha
Copy link
Member

produce a warning if they are - this is because encoding a package name is mostly undefined behaviour that the scala 3 compiler magically recovers from - but could be problematic.

fixes #14448

@bishabosha
Copy link
Member Author

test performance please

compiler/src/dotty/tools/dotc/ast/Desugar.scala Outdated Show resolved Hide resolved
compiler/src/dotty/tools/dotc/core/NameOps.scala Outdated Show resolved Hide resolved
compiler/src/dotty/tools/dotc/util/Chars.scala Outdated Show resolved Hide resolved
@bishabosha bishabosha force-pushed the warn-symbolic-package branch 2 times, most recently from d7e4cfa to bde2c9e Compare March 2, 2022 09:09
produce a warning if they are - this is
because encoding a package name is mostly
undefined behaviour that the scala 3 compiler
magically recovers from - but could be problematic.
@bishabosha bishabosha added the needs-minor-release This PR cannot be merged until the next minor release label Mar 2, 2022
@bishabosha bishabosha added this to the 3.2.0-RC1 milestone Mar 2, 2022
@bishabosha bishabosha removed the needs-minor-release This PR cannot be merged until the next minor release label Mar 7, 2022
@bishabosha bishabosha modified the milestones: 3.2.0-RC1, 3.1.3-RC1 Mar 7, 2022
@bishabosha bishabosha merged commit 6dc591a into scala:main Mar 7, 2022
@bishabosha bishabosha deleted the warn-symbolic-package branch March 7, 2022 15:01
@Kordyjan Kordyjan modified the milestones: 3.1.3-RC1, 3.1.3 Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

In TASTy, a symbolic package can either be encoded or unencoded
3 participants