RFR: 8295153: java/util/Base64/TestEncodingDecodingLength.java ran out of memory [v2]
Justin Lu
jlu at openjdk.org
Wed May 1 18:52:06 UTC 2024
On Wed, 1 May 2024 16:40:31 GMT, Lance Andersen <lancea at openjdk.org> wrote:
>> Justin Lu has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Reflect review comments
>
> test/jdk/java/util/Base64/TestEncodingDecodingLength.java line 47:
>
>> 45: public class TestEncodingDecodingLength {
>> 46:
>> 47: private static final int size = Integer.MAX_VALUE - 8;
>
> Perhaps consider size-> SIZE and maybe tweak the name and add an additional comment to its purpose
Thanks for the review Lance, both comments should be addressed in https://github.com/openjdk/jdk/pull/19036/commits/aa25c9a72d218401f70bb19bd0e1d911b19b6361
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19036#discussion_r1586643285
More information about the core-libs-dev
mailing list