RFR(S): 8221262: Cleanups in UnixFileSystem/WinNTFileSystem implementation classes

Langer, Christoph christoph.langer at sap.com
Sun Mar 24 20:26:14 UTC 2019


Hi Ivan,

thanks for looking closely.

> UnixFileSystem_md.c:
> Should the second error message be "Could not close file"?

Yes, maybe that's better. Though it's really just the fallback, one could at least distinguish the two possibilities of IOException in this method...

> UnixFileSystem.java:
> Line spacing looks inconsistent.  Please leave an empty line before
> @Override at lines 267, 269, 271, 273, etc.

Good point, I compared to WinNTFileSystem.java and aligned the line spacing.

Please find the updated webrev here:
http://cr.openjdk.java.net/~clanger/webrevs/8221262.1/

Best regards
Christoph



More information about the core-libs-dev mailing list