Commitd5cc7bc

Luis Garza

authored and

committed

Fixed Java 11+ compatibility issue by improving exception handling in StreamWrapper.finalize()

1 parent90ee03c commitd5cc7bcCopy full SHA for d5cc7bc

File tree

+11

-1

lines changed

+11

-1

lines changed

Lines changed: 11 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -113,11 +113,21 @@ public void close() throws IOException {`
`113`	`113`	`}`
`114`	`114`	`}`
`115`	`115`
	`116`	`+// Suppress deprecation warning: finalize() is deprecated in Java 9+ but needed`
	`117`	`+// for Java 8 compatibility and reliable resource cleanup in this specific case`
	`118`	`+@SuppressWarnings("deprecation")`
	`119`	`+@Override`
`116`	`120`	`protectedvoidfinalize()throwsIOException {`
`117`	`121`	`// forcibly close it because super.finalize() may keep the FD open, which may prevent`
`118`	`122`	`// file deletion`
`119`	`123`	`close();`
`120`		`-super.finalize();`
	`124`	`+try {`
	`125`	`+super.finalize();`
	`126`	`+ }catch (Throwablet) {`
	`127`	`+// Ignore exceptions during finalization - object is being garbage collected`
	`128`	`+// and no recovery is possible. This also handles Java 11+ compatibility where`
	`129`	`+// super.finalize() throws Throwable instead of just IOException`
	`130`	`+ }`
`121`	`131`	`}`
`122`	`132`	`};`
`123`	`133`	`}else {`

Comments

(0)