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

Fixing bug in global initialization checker when accessing global object #20424

Closed
wants to merge 3 commits into from

Conversation

EnzeXing
Copy link
Contributor

@EnzeXing EnzeXing commented May 17, 2024

This PR fixes a bug in the global initialization checker such that when accessing global object through full path, the checker immediately analyzes the initialization code of the corresponding object. Also adds test-cases for cyclic initialization causing deadlock

[test_scala2_library_tasty]

For some reason, BestEffortTastyWriter does not compile since PlainFile does not have a delete
method, only File does.
@liufengyun
Copy link
Contributor

@EnzeXing LGTM

To avoid breaking the nightly build, I enabled test_scala2_library_tasty. Could you please rebase to trigger the test?

@EnzeXing EnzeXing force-pushed the extend-whitelist branch 3 times, most recently from 3aebcac to 81c5c57 Compare May 27, 2024 14:32
@EnzeXing EnzeXing force-pushed the extend-whitelist branch from 81c5c57 to ae74dcf Compare May 27, 2024 14:38
olhotak added a commit that referenced this pull request Sep 3, 2024
This PR is an updated version of #20424, which fixes a bug in the global
initialization checker such that when accessing global object through
full path, the checker immediately analyzes the initialization code of
the corresponding object.

[test_scala2_library_tasty]
@olhotak
Copy link
Contributor

olhotak commented Sep 3, 2024

Closing in favour of #21511

@olhotak olhotak closed this Sep 3, 2024
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