RFR [XS] : 8234339: replace JLI_StrTok in java_md_solinux.c

Roger Riggs Roger.Riggs at oracle.com
Wed Nov 20 15:07:58 UTC 2019


Hi Matthias,

Good to see the switch to strtok_r.
Looks fine.

Thanks, Roger


On 11/19/19 4:23 AM, Baesken, Matthias wrote:
> Hello, please review this small change .
>
> JLI_StrTok is only used in one function, so the define can be replaced, probably we should use the "safer" strtok_r (the MT safety might not be a big deal in the launcher however).
>
>
> Bug/webrev :
>
> https://bugs.openjdk.java.net/browse/JDK-8234339
>
> http://cr.openjdk.java.net/~mbaesken/webrevs/8234339.0/
>
>
> Thanks, Matthias



More information about the core-libs-dev mailing list