Skip to content

CoreASTProvider: avoid Stackoverflow #1977 #2032

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jukzi
Copy link
Contributor

@jukzi jukzi commented Feb 19, 2025

replaces tail recursion with infinite loop

#1977

@jukzi jukzi added the bug Something isn't working label Feb 19, 2025
@jukzi jukzi force-pushed the CoreASTProvider_avoid_Stackoverflow_ branch from b8ea18e to 1874aab Compare February 19, 2025 10:11
@jukzi
Copy link
Contributor Author

jukzi commented Feb 19, 2025

Note that this fix only solves the Stackoverflow when waiting too long. It does not try to solve deadlocks that have been reported in https://bugs.eclipse.org/bugs/show_bug.cgi?id=366048

@jukzi jukzi force-pushed the CoreASTProvider_avoid_Stackoverflow_ branch from 1874aab to c1cccd3 Compare March 24, 2025 08:38
replaces tail recursion with infinite loop

eclipse-jdt#1977
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants