Thanks for pressing on that, Viktor!
I think I was fooled by my quick-and-dirty test. As I tried to harden it, I
failed to deduce consistent behavior about what parallel stream does when
unchecked exceptions happen. And in fact, it seems like it does *not*
interrupt pending threads (?).
With this
On Sat, 12 Jul 2025 17:36:54 GMT, Matthias Baesken wrote:
>> The following is reported when building with the gcc static analyzer
>> (-fanalyzer) :
>>
>>
>> /jdk/src/java.base/unix/native/libjava/java_props_md.c:244:17: warning: leak
>> of 'temp' [CWE-401] [-Wanalyzer-malloc-leak]
>> 244 |
> The following is reported when building with the gcc static analyzer
> (-fanalyzer) :
>
>
> /jdk/src/java.base/unix/native/libjava/java_props_md.c:244:17: warning: leak
> of 'temp' [CWE-401] [-Wanalyzer-malloc-leak]
> 244 | return 0;
>
>
> Seems we have to free temp in an early return.
M
On Fri, 11 Jul 2025 13:04:37 GMT, Aleksey Shipilev wrote:
> There is another instance a few lines below. See around where
> `free(encoding_variant);` happen.
Hi I noticed that one too ! Seems we have to adjust it too , I first thought
that it is not needed there but it is.
Not sure why the G
> This PR implements nth root computation for BigIntegers using Newton method.
fabioromano1 has updated the pull request incrementally with one additional
commit since the last revision:
Removed useless instruction
-
Changes:
- all: https://git.openjdk.org/jdk/pull/24898/files
> This PR implements nth root computation for BigIntegers using Newton method.
fabioromano1 has updated the pull request incrementally with one additional
commit since the last revision:
Code simplification
-
Changes:
- all: https://git.openjdk.org/jdk/pull/24898/files
- new:
