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

System.out and System.err collision causing deadlock #338

Closed
processing-bot opened this issue Jan 11, 2022 · 7 comments
Closed

System.out and System.err collision causing deadlock #338

processing-bot opened this issue Jan 11, 2022 · 7 comments

Comments

@processing-bot
Copy link
Collaborator

Created by: benfry

Report for stdout/stderr collisions causing an error:

I linked to the start of the important changes in the app/src/processing/app/ui/EditorConsole.java file, but the links can be buggy; link to final file. (I had to squash the files when the processing 4 history was rewritten to remove large files; probably could've avoided doing that, but didn't bother). for (int i = 0; i < 10000; i++) { println("out"); System.err.println("err"); } quickly results in some java.util.EmptyStackExceptions being thrown and running being broken.

Originally posted by @dzaima in processing/processing#6230 (comment)

@processing-bot
Copy link
Collaborator Author

Created by: benfry

@dzaima does that code actually contain a fix? (From your earlier message it sounded like it was a failed attempt.)

@processing-bot
Copy link
Collaborator Author

Created by: dzaima

I've ran it in an infinite loop for a couple minutes and it hasn't broken (whereas previously it fails in less than a second), so.. probably.

@processing-bot
Copy link
Collaborator Author

Created by: dzaima

(also, previous discussion and another one)

@processing-bot
Copy link
Collaborator Author

Created by: benfry

That still caused the problem for me, but I've just checked in a fix that works (at least on this end). With any luck, it should be resolved for 4.0 beta 3.

@processing-bot
Copy link
Collaborator Author

Created by: dzaima

Can confirm that fix works for me!

@processing-bot
Copy link
Collaborator Author

Created by: benfry

Thanks for checking, and for the links to those other reports.

@processing-bot
Copy link
Collaborator Author

Created by: github-actions[bot]

This issue has been automatically locked. To avoid confusion with reports that have already been resolved, closed issues are automatically locked 30 days after the last comment. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant