Re: RFR 8180349: Review JAXP Java SE 9 API javadocs

2017-05-23 Thread huizhe wang
On 5/23/2017 3:05 PM, Lance Andersen wrote: Hi Joe, Looks good overall and one comment below on top of Roger’s: On May 23, 2017, at 4:38 PM, Roger Riggs wrote: Hi Joe, Looks good, a few minor editorial suggestions: javax/transform/sax/package-info.java: - line 28: I don't think the '-'

Re: RFR 8180349: Review JAXP Java SE 9 API javadocs

2017-05-23 Thread Lance Andersen
Hi Joe, Looks good overall and one comment below on top of Roger’s: > On May 23, 2017, at 4:38 PM, Roger Riggs wrote: > > Hi Joe, > > Looks good, a few minor editorial suggestions: > > javax/transform/sax/package-info.java: > - line 28: I don't think the '-' belongs after SAX2 in the first s

Re: RFR 8180349: Review JAXP Java SE 9 API javadocs

2017-05-23 Thread Roger Riggs
Hi Joe, Looks good, a few minor editorial suggestions: javax/transform/sax/package-info.java: - line 28: I don't think the '-' belongs after SAX2 in the first sentence. javax/transform/dom/package-info.java: line 28: ditto it is more readable without the '-'. javax/stream/package-info.ja

RFR 8180349: Review JAXP Java SE 9 API javadocs

2017-05-23 Thread huizhe wang
Hi, Please review an update to the JAXP Java SE 9 API javadocs. JBS: https://bugs.openjdk.java.net/browse/JDK-8180349 webrevs: http://cr.openjdk.java.net/~joehw/jdk9/8180349/webrev/ There's a lot of format changes while converting to package-info.java, but only minor (wording) changes to the o