Open jukzi opened 1 week ago
The trivial solution would be to remove the circular method call to read() in line 65. But this would also remove the workaround mentioned in the class comment:
Workaround problem with input stream from a <code>java.lang.Process</code> that throws an <code>IOException</code> even if there is something to read.
This workaround was introduced 22 years ago and maybe it is no longer necessary today. Perhaps we could simply remove the whole class org.eclipse.jdt.core.tests.runtime.VMInputStream
.
its only a test, but such recursion might lead to stackoverflow:
https://download.eclipse.org/eclipse/downloads/drops4/I20241111-1800/testresults/consolelogs/ep434I-unit-linux-x86_64-java23_linux.gtk.x86_64_23_consolelog.txt