Change looks fine. BTW, have you run the test with that special locale on?
Thanks Max > On Jun 22, 2016, at 2:50 PM, Tim Du <[email protected]> wrote: > > Hi, > > Would you please review the path for following bug? > > bug: https://bugs.openjdk.java.net/browse/JDK-8146393 > webrev: http://cr.openjdk.java.net/~tidu/8146393/webrev.00/ > > The test failed when using String.format() to format the server port on ar_SA > Locale, Adding Locale.ROOT arguments to avoid the locale sensitive > operations. It could pass on ar_SA locale platform after this code > change.Thanks > > Regards > Tim
