Hi Christoph / Roger , thanks for the reviews ! Best regards , Matthias
> -----Original Message----- > From: Langer, Christoph <[email protected]> > Sent: Donnerstag, 21. November 2019 23:13 > To: Roger Riggs <[email protected]>; core-libs- > [email protected]; Baesken, Matthias <[email protected]> > Subject: RE: RFR [XS] : 8234339: replace JLI_StrTok in java_md_solinux.c > > +1 > > > -----Original Message----- > > From: core-libs-dev <[email protected]> On Behalf > > Of Roger Riggs > > Sent: Mittwoch, 20. November 2019 16:08 > > To: [email protected] > > Subject: Re: RFR [XS] : 8234339: replace JLI_StrTok in java_md_solinux.c > > > > 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
