On Mon, 5 May 2025 20:58:56 GMT, Andrey Turbanov wrote:
>> Raffaello Giulietti has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Increased min heap size to 8G.
>
> Marked as reviewed by aturbanov (Committer).
Thanks to the reviewers, and
On Mon, 5 May 2025 17:29:33 GMT, Raffaello Giulietti
wrote:
>> A fix to throw `OutOfMemoryError`, as done in releases ≤ 23.
>
> Raffaello Giulietti has updated the pull request incrementally with one
> additional commit since the last revision:
>
> Increased min heap size to 8G.
Looks good,
On Tue, 6 May 2025 08:44:56 GMT, Raffaello Giulietti
wrote:
>> @ForceInline
>> static int checkOverflow(int value) {
>> if (value >= 0) {
>> return value;
>> }
>> throw new OutOfMemoryError("Overflow: String length out of range");
>> }
>>
>> priva
On Tue, 6 May 2025 08:22:51 GMT, Shaojin Wen wrote:
>> src/java.base/share/classes/java/lang/StringConcatHelper.java line 435:
>>
>>> 433: static String doConcat(String s1, String s2) {
>>> 434: byte coder = (byte) (s1.coder() | s2.coder());
>>> 435: int newLength = checkOver
On Tue, 6 May 2025 07:44:29 GMT, Claes Redestad wrote:
>> Raffaello Giulietti has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Increased min heap size to 8G.
>
> src/java.base/share/classes/java/lang/StringConcatHelper.java line 435:
>
>
On Mon, 5 May 2025 17:29:33 GMT, Raffaello Giulietti
wrote:
>> A fix to throw `OutOfMemoryError`, as done in releases ≤ 23.
>
> Raffaello Giulietti has updated the pull request incrementally with one
> additional commit since the last revision:
>
> Increased min heap size to 8G.
LGTM
src/j
On Mon, 5 May 2025 17:29:33 GMT, Raffaello Giulietti
wrote:
>> A fix to throw `OutOfMemoryError`, as done in releases ≤ 23.
>
> Raffaello Giulietti has updated the pull request incrementally with one
> additional commit since the last revision:
>
> Increased min heap size to 8G.
Marked as r
On Mon, 5 May 2025 17:29:33 GMT, Raffaello Giulietti
wrote:
>> A fix to throw `OutOfMemoryError`, as done in releases ≤ 23.
>
> Raffaello Giulietti has updated the pull request incrementally with one
> additional commit since the last revision:
>
> Increased min heap size to 8G.
Test update
> A fix to throw `OutOfMemoryError`, as done in releases ≤ 23.
Raffaello Giulietti has updated the pull request incrementally with one
additional commit since the last revision:
Increased min heap size to 8G.
-
Changes:
- all: https://git.openjdk.org/jdk/pull/25038/files
- ne
