Re: RFR:JDK-8066806:java.time.format.DateTimeFormatter cannot parse an offset with single digit hour

2016-07-24 Thread nadeesh tv
Hi Stephen, Thanks for the comments. Please see the updated webrev http://cr.openjdk.java.net/~ntv/8066806/webrev.11/ Changes: Included the suggestions of Stephen Thanks and regards, Nadeesh On 7/22/2016 3:38 PM, Stephen Colebourne wrote: These tests are expected to throw exceptions: tes

Re: RFR (JAXP) JDK-8067170: Enable security manager on JAXP unit tests

2016-07-24 Thread huizhe wang
On 7/22/2016 5:53 AM, Daniel Fuchs wrote: On 22/07/16 10:15, Frank Yuan wrote: Hi Daniel Thank you very much for your review and the comments! -Original Message- From: Daniel Fuchs [mailto:daniel.fu...@oracle.com] Subject: Re: RFR (JAXP) JDK-8067170: Enable security manager on JAXP

RFR (JAXP) 8158084: Catalog API: JAXP XML Processor Support

2016-07-24 Thread huizhe wang
Hi, This is an enhancement to support the Catalog API (javax.xml.catalog) throughout the JAXP processors. All of the CatalogFeatures are supported by the JAXP factories and processors (e.g. SAXParser), and corresponding System properties as specified in the CatalogFeatures. An additional java

Re: [8u-dev] RFR (JAXP) 8028363: XmlGregorianCalendarImpl.getTimeZone() bug when offset is less than 10 minutes

2016-07-24 Thread huizhe wang
+1. -Joe On 7/21/2016 10:40 AM, Seán Coffey wrote: Looks fine to me also Svetlana. Regards, Sean. On 21/07/2016 13:13, Svetlana Nikandrova wrote: Little reminder. On 19.07.2016 19:29, Svetlana Nikandrova wrote: Hello, please review this fix for 8u-dev. For jdk9 problem has been addressed

Re: [PATCH] JDK-8155102: Process.toString could include pid, isAlive, exitStatus

2016-07-24 Thread Andrey Dyachkov
Hi Roger, Thank you for reviewing it! I've prepared another version. diff --git a/src/java.base/unix/classes/java/lang/ProcessImpl.java b/src/java.base/unix/classes/java/lang/ProcessImpl.java --- a/src/java.base/unix/classes/java/lang/ProcessImpl.java +++ b/src/java.base/unix/classes/java/lang/Pr