RFR: 8339475: Check return value of pthread_create in jdk coding
Christoph Langer
clanger at openjdk.org
Thu Sep 26 06:57:36 UTC 2024
On Wed, 25 Sep 2024 11:46:00 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:
> It has been discussed that checking the return value of pthread_create should be done.
> See the discussion here https://github.com/openjdk/jdk/pull/20812 about the splashscreen coding .
Looks good overall, just one nit.
I guess it's ok to print a warning to stderr in this case.
src/java.desktop/macosx/native/libsplashscreen/splashscreen_sys.m line 278:
> 276: int rslt = pthread_attr_init(&attr);
> 277: if (rslt != 0) return;
> 278: rc = pthread_create(&thr, &attr, SplashScreenThread, (void *) splash);
For the sake of symmetry, could you rename rc to rslt here?
-------------
Changes requested by clanger (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/21182#pullrequestreview-2330276303
PR Review Comment: https://git.openjdk.org/jdk/pull/21182#discussion_r1776482707
More information about the client-libs-dev
mailing list