Hi Igor,
In SATestUtils.java you do
var bb = ... Files.readAllBytes(...) ...
and then use bb[0]
if the file has 0 length, old code throws EOFException and new one will
throw IndexOutOfBoundsException.
And looks like the caller doesn't expect it (it catches IOException).
--alex
On 06/09/2020 16:47, Igor Ignatyev wrote:
http://cr.openjdk.java.net/~iignatyev//8183040/webrev.00
38 lines changed: 8 ins; 16 del; 14 mod;
Hi all,
could you please review this small clean up of testlibrary classes which
updates j.t.lib.Platform and j.t.l.SA.SATestUtils (as it now contains the
methods which 8183040 was about) to use NIO file API?
testing: test/hotspot/jtreg/serviceability
webrev: http://cr.openjdk.java.net/~iignatyev//8183040/webrev.00
JBS: https://bugs.openjdk.java.net/browse/JDK-8183040
Thanks,
-- Igor
