Skip to content

Workaround LazyRef CyclicReference in genDocs #8077

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

Conversation

nicolasstucki
Copy link
Contributor

Add more explicit types to avoid inference involving type matches.
Also avoid recomputing tuple sizes.

This issue was present in both https://dotty-ci.epfl.ch/lampepfl/dotty/3729 and https://dotty-ci.epfl.ch/lampepfl/dotty/3730.

Add more explicit types to avoid inference involving type matches.
Also avoid recomputing tuple sizes.
@nicolasstucki
Copy link
Contributor Author

Though UnsafeExpr.scala what involved in one of the merged PRs, it was not the only file that needed to be updated.

@nicolasstucki
Copy link
Contributor Author

I saw in some cases some stack overflows that may have been caught and converted into CyclicReference errors.

@nicolasstucki nicolasstucki added fasttrack Simple fix. Reviewer should merge or apply additional changes directly. prio:high labels Jan 23, 2020
@anatoliykmetyuk anatoliykmetyuk merged commit b96a026 into scala:master Jan 23, 2020
@anatoliykmetyuk anatoliykmetyuk deleted the workaround-lazy-ref-ciclic-ref-in-genDocs branch January 23, 2020 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fasttrack Simple fix. Reviewer should merge or apply additional changes directly. prio:high
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants