-
-
Notifications
You must be signed in to change notification settings - Fork 647
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
[wip] Handle infinite recursion in Std.string()
#8377
base: development
Are you sure you want to change the base?
[wip] Handle infinite recursion in Std.string()
#8377
Conversation
Std.string()
Std.string()
Could you update the branch please? |
It is standard behavior in other languages to throw a Stack Overflow Exception when, for instance, trying to stringify a self-referencing object. I don't see why this is bug, nor why this should be "fixed"... But whatever, if this were to be implemented, the default recursion limit should DEFINITELY be much higher than 5. |
Getting stack overflows from recursive value printing is not a good user experience. We should definitely account for that. As for the limit, I agree that the 5 is a bit low. We can increase that and maybe make it configurable through a define. Also, the branch needs an update. |
Fixes #6728