Hi Volker, thanks for looking at this change!
> Looks good, but don't we also need this for getIntValue() as well? This class is a helper class for testing jstat. To my knowledge jstat never formats integers, so the current parsing should cover all possible outputs to be tested. Best regards, Goetz. > -----Original Message----- > From: Volker Simonis [mailto:volker.simo...@gmail.com] > Sent: Thursday, July 27, 2017 11:49 AM > To: Lindenmaier, Goetz <goetz.lindenma...@sap.com> > Cc: serviceability-dev (serviceability-dev@openjdk.java.net) <serviceability- > d...@openjdk.java.net>; Zeller, Arno <arno.zel...@sap.com> > Subject: Re: RFR(XS): 8185112: [TESTBUG] Servicability tests cannot parse > float if non US locale. > > Looks good, but don't we also need this for getIntValue() as well? > I.e. can't an integer be "1.234.678" (German style) as well as > "1,234,678" (American style) for example ? > > Thanks, > Volker > > On Mon, Jul 24, 2017 at 9:07 AM, Lindenmaier, Goetz > <goetz.lindenma...@sap.com> wrote: > > Hi, > > > > > > > > Some tests use Float.valueOf for String to float converting. If an other > > locale than US is used the test failed. We observed this on Mac. > > Changed to use NumberFormat to work with all locales. > > > > > > > > Please review this change. I please need a sponsor. > > > > http://cr.openjdk.java.net/~goetz/wr17/8185112-macLocale/webrev.01/ > > > > > > > > Best regards, > > > > Goetz