[crac] RFR: Improved open file descriptors tracking [v9]

Radim Vansa duke at openjdk.org
Tue May 9 15:38:11 UTC 2023


On Tue, 2 May 2023 17:00:11 GMT, Anton Kozlov <akozlov at openjdk.org> wrote:

>> Radim Vansa has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Provide more information for file descriptors
>
> src/java.base/unix/native/libnet/SocketImpl.c line 115:
> 
>> 113:            return NULL;
>> 114:         }
>> 115:     }
> 
> No need for `} else {` here and everywhere else since the previous block has anyway terminated the function.
> 
> This will make the code more streamlined.
> 
> Suggestion:
> 
> }
> 
> localAddr = create_isa(env, isa_class, isa_ctor, &local);
> if (localAddr == NULL) {
>     JNU_ThrowOutOfMemoryError(env, "java.net.InetSocketAddres");
>     return NULL;
> }

The code was supposed to mirror closer the remote part where we can't do that, but OK.

-------------

PR Review Comment: https://git.openjdk.org/crac/pull/43#discussion_r1188790339


More information about the crac-dev mailing list