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

2019-11-22 Thread Baesken, Matthias
: 8234339: replace JLI_StrTok in java_md_solinux.c > > +1 > > > -Original Message- > > From: core-libs-dev On Behalf > > Of Roger Riggs > > Sent: Mittwoch, 20. November 2019 16:08 > > To: core-libs-dev@openjdk.java.net > > Subject: Re: RFR [XS

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

2019-11-21 Thread Langer, Christoph
+1 > -Original Message- > From: core-libs-dev On Behalf > Of Roger Riggs > Sent: Mittwoch, 20. November 2019 16:08 > To: core-libs-dev@openjdk.java.net > Subject: Re: RFR [XS] : 8234339: replace JLI_StrTok in java_md_solinux.c > > Hi Matthias, > > Good

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

2019-11-20 Thread Roger Riggs
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 safet