Skip to content

Context escape detection. #119

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 3 commits into from
May 6, 2014
Merged

Context escape detection. #119

merged 3 commits into from
May 6, 2014

Conversation

DarkDimius
Copy link
Contributor

During creation of each of DottyTests context is stolen from test and a
WeakReference for it is created.
After running all tests references are examined to detect if any of them
has leaked.

During creation of each of DottyTests context is stolen from test and a
WeakReference for it is created.
After running all tests references are examined to detect if any of them
has leaked.
@DarkDimius
Copy link
Contributor Author

@odersky @gzm0 please have a look.

@DarkDimius
Copy link
Contributor Author

While playing with backend I've found that this is not enough.
BaseContextes are also created in Drivers(which are Main and Bench used in tests) from scratch, and this patch doesn't address this

@DarkDimius
Copy link
Contributor Author

@odersky please have a look on new commit

@DarkDimius
Copy link
Contributor Author

@gzm0 can you please have a look?

@gzm0
Copy link
Contributor

gzm0 commented May 5, 2014

After discussion with @DarkDimius, we realized that SBT currently doesn't run stuff in parallel (don't know why exactly). So you are cleared to merge.

@gzm0
Copy link
Contributor

gzm0 commented May 5, 2014

LGTM

DarkDimius added a commit that referenced this pull request May 6, 2014
Context escape detection.
@DarkDimius DarkDimius merged commit bf9ae99 into scala:master May 6, 2014
WojciechMazur pushed a commit to WojciechMazur/dotty that referenced this pull request Mar 19, 2025
Backport "Nowarn extension matching nonpublic member" to 3.3 LTS
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.

4 participants