RFR  BufferedInputStream calculates negative array size with large streams and mark
ivan.gerasimov at oracle.com
Thu Aug 22 13:52:30 UTC 2013
Thank you Alan!
Would you please review the updated webrev?
On 21.08.2013 13:28, Alan Bateman wrote:
> On 20/08/2013 17:01, Ivan Gerasimov wrote:
>> Yes, you're right!
>> I've updated the test to be a shell script + Java application.
>> If java reported 'Invalid maximum heap size', the test is skipped.
>> Just check with JPRT, the test passes on both 64 and 32 bit platforms.
>> Would you please review the updated webrev?
> The updated change to BufferedInputStream looks good.
> On the test then I agree with the approach to skip on 32-bit but I
> think we should avoid introduce more shell tests. Have you looked at
> jdk.testlibrary.ProcessTools to launch the test after checking that we
> are 64-bit?
That's a good alternative to shell scripting, I like it.
More information about the core-libs-dev