Hi Volker, > OK, I don't want to unnecessarily block this change but IMO if we > don't need the integer parsing function at all we should remove it or > otherwise change it to use NumberFormat as well. We need the integer parsing function. But the intergers parsed are not printed with formatting. Just plain 123456. So I don't see how the locale will interfere. But I changed it anyways (took a while because the tests didn't run nightly): http://cr.openjdk.java.net/~goetz/wr17/8185112-macLocale/webrev.02/
Best regards, Goetz. > -----Original Message----- > From: Volker Simonis [mailto:[email protected]] > Sent: Freitag, 28. Juli 2017 16:45 > To: Lindenmaier, Goetz <[email protected]> > Cc: serviceability-dev ([email protected]) <serviceability- > [email protected]>; Zeller, Arno <[email protected]> > Subject: Re: RFR(XS): 8185112: [TESTBUG] Servicability tests cannot parse > float if non US locale. > > On Fri, Jul 28, 2017 at 9:30 AM, Lindenmaier, Goetz > <[email protected]> wrote: > > 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. > > > > OK, I don't want to unnecessarily block this change but IMO if we > don't need the integer parsing function at all we should remove it or > otherwise change it to use NumberFormat as well. > > Regards, > Volker > > > Best regards, > > Goetz. > > > >> -----Original Message----- > >> From: Volker Simonis [mailto:[email protected]] > >> Sent: Thursday, July 27, 2017 11:49 AM > >> To: Lindenmaier, Goetz <[email protected]> > >> Cc: serviceability-dev ([email protected]) > <serviceability- > >> [email protected]>; Zeller, Arno <[email protected]> > >> 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 > >> <[email protected]> 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
