RE: RFR(s) 8170153: PPC64: Poor StrictMath performance due to non-optimized compilation

2016-11-24 Thread Doerr, Martin
Hi Gustavo, thanks for providing the webrevs. I have ran the StrictMath jck tests which fail when building with -O3 and without -ffp-contract=off: FailedTests: api/java_lang/StrictMath/desc.html#acos javasoft.sqe.tests.api.java.lang.StrictMath.acos_test api/java_lang/StrictMath/desc.html#asin

RFR: JDK-8167648: java.io.PrintWriter should have PrintWriter((String|File), Charset) constructors

2016-11-24 Thread Patrick Reinhart
Any sponsor for this? > Am 23.11.2016 um 21:53 schrieb Patrick Reinhart : > > Hi Stephen, > > I changed the webrev accordingly: > > http://cr.openjdk.java.net/~reinhapa/reviews/8167648/webrev.00 > > -Patrick > >> Am 23.11.2016 um 14:45 schrieb Stephen Colebourne : >> >> Returning the writer

Re: RFR JDK-8170192 [JAXP] [TESTBUG] test/javax/xml/jaxp/libs/jaxp/library/JAXPPolicyManager.java should grant permissions to jtreg, javatest, and testng jars

2016-11-24 Thread Lance Andersen
Looks good Frank Best Lance > On Nov 23, 2016, at 9:29 PM, Frank Yuan wrote: > > Hi Jon > > > > Thank you for your advice! > > > > Please check the update http://cr.openjdk.java.net/~fyuan/8170192/webrev.01/ > , which contains jcommander.jar and removes the extra > blank lines following C

Re: RFR JDK-8170192 [JAXP] [TESTBUG] test/javax/xml/jaxp/libs/jaxp/library/JAXPPolicyManager.java should grant permissions to jtreg, javatest, and testng jars

2016-11-24 Thread Daniel Fuchs
Hi Frank, Looks good to me. Thanks for the advice Jon! best regards, -- daniel On 24/11/16 02:29, Frank Yuan wrote: Hi Jon Thank you for your advice! Please check the update http://cr.openjdk.java.net/~fyuan/8170192/webrev.01/ , which contains jcommander.jar and removes the extra blank