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
