Re: RFR(XS) : 8186095 : upgrade to jtreg 4.2 b08

2017-08-10 Thread David Holmes
Thanks Ioi, and Igor, for clarifying. David On 11/08/2017 3:23 PM, Ioi Lam wrote: On 8/10/17 9:46 PM, David Holmes wrote: On 11/08/2017 2:31 PM, Igor Ignatyev wrote: On Aug 10, 2017, at 9:22 PM, David Holmes wrote: Hi Igor, On 11/08/2017 2:02 PM, Igor Ignatyev wrote: http://cr.openj

Re: RFR(XS) : 8186095 : upgrade to jtreg 4.2 b08

2017-08-10 Thread Ioi Lam
On 8/10/17 9:46 PM, David Holmes wrote: On 11/08/2017 2:31 PM, Igor Ignatyev wrote: On Aug 10, 2017, at 9:22 PM, David Holmes wrote: Hi Igor, On 11/08/2017 2:02 PM, Igor Ignatyev wrote: http://cr.openjdk.java.net/~iignatyev//8186095/webrev.00/index.html 14 lines changed: 1 ins; 0 del;

Re: RFR(XS) : 8186095 : upgrade to jtreg 4.2 b08

2017-08-10 Thread Igor Ignatyev
> On Aug 10, 2017, at 9:46 PM, David Holmes wrote: > > On 11/08/2017 2:31 PM, Igor Ignatyev wrote: >>> On Aug 10, 2017, at 9:22 PM, David Holmes wrote: >>> >>> Hi Igor, >>> >>> On 11/08/2017 2:02 PM, Igor Ignatyev wrote: http://cr.openjdk.java.net/~iignatyev//8186095/webrev.00/index.html

Re: RFR(XS) : 8186095 : upgrade to jtreg 4.2 b08

2017-08-10 Thread David Holmes
On 11/08/2017 2:31 PM, Igor Ignatyev wrote: On Aug 10, 2017, at 9:22 PM, David Holmes wrote: Hi Igor, On 11/08/2017 2:02 PM, Igor Ignatyev wrote: http://cr.openjdk.java.net/~iignatyev//8186095/webrev.00/index.html 14 lines changed: 1 ins; 0 del; 13 mod; Hi all, could you please review thi

Re: RFR(XS) : 8186095 : upgrade to jtreg 4.2 b08

2017-08-10 Thread Igor Ignatyev
> On Aug 10, 2017, at 9:22 PM, David Holmes wrote: > > Hi Igor, > > On 11/08/2017 2:02 PM, Igor Ignatyev wrote: >> http://cr.openjdk.java.net/~iignatyev//8186095/webrev.00/index.html >>> 14 lines changed: 1 ins; 0 del; 13 mod; >> Hi all, >> could you please review this small patch which bumps u

Re: RFR(XS) : 8186095 : upgrade to jtreg 4.2 b08

2017-08-10 Thread David Holmes
Hi Igor, On 11/08/2017 2:02 PM, Igor Ignatyev wrote: http://cr.openjdk.java.net/~iignatyev//8186095/webrev.00/index.html 14 lines changed: 1 ins; 0 del; 13 mod; Hi all, could you please review this small patch which bumps up jtreg version? besides updating jib profiles and all TEST.ROOT file

RFR(XS) : 8186095 : upgrade to jtreg 4.2 b08

2017-08-10 Thread Igor Ignatyev
http://cr.openjdk.java.net/~iignatyev//8186095/webrev.00/index.html > 14 lines changed: 1 ins; 0 del; 13 mod; Hi all, could you please review this small patch which bumps up jtreg version? besides updating jib profiles and all TEST.ROOT files, the fix updates hotspot/test/runtime/Metaspace/Frag

Re: RFR: JDK-8186052: Fix a11y and HTML issues in the java.base/java.lang[.*] packages

2017-08-10 Thread Jonathan Gibbons
Update: addresses Stuart's comment for String.split Webrev: http://cr.openjdk.java.net/~jjg/8186052/webrev.01/index.html API: http://cr.openjdk.java.net/~jjg/8186052/api.01/index.html -- Jon On 08/09/2017 06:28 PM, Stuart Marks wrote: On 8/9/17 5:11 PM, Jonathan Gibbons wrote: Please review th

Re: RFR: JDK-8185994: Fix a11y and HTML issues in the java.base/java.io and java.base/java.nio packages

2017-08-10 Thread Brian Burkhalter
I think it looks fine now. Brian On Aug 10, 2017, at 1:59 PM, Jonathan Gibbons wrote: > I have posted an update to the webrev and API. > > The change is in DataInput ... I have tweaked the position of the text in the > cells of that initial table. > I would prefer not to iterate further on t

Re: RFR: JDK-8185994: Fix a11y and HTML issues in the java.base/java.io and java.base/java.nio packages

2017-08-10 Thread Jonathan Gibbons
Ooops ... forgot the updated links: Webrev: http://cr.openjdk.java.net/~jjg/8185994/webrev.01/index.html API: http://cr.openjdk.java.net/~jjg/8185994/api.01/index.html -- Jon On 08/10/2017 01:59 PM, Jonathan Gibbons wrote: I have posted an update to the webrev and API. The change is in DataIn

Re: RFR: JDK-8185994: Fix a11y and HTML issues in the java.base/java.io and java.base/java.nio packages

2017-08-10 Thread Jonathan Gibbons
I have posted an update to the webrev and API. The change is in DataInput ... I have tweaked the position of the text in the cells of that initial table. I would prefer not to iterate further on the layout of this table, since the changeset is primarily about addressing accessibility and HTML

Re: RFR: JDK-8185994: Fix a11y and HTML issues in the java.base/java.io and java.base/java.nio packages

2017-08-10 Thread Brian Burkhalter
On Aug 9, 2017, at 6:56 PM, Stuart Marks wrote: > On 8/9/17 5:14 PM, Jonathan Gibbons wrote: >> Please review this proposed change to address accessibility and HTML issues >> in the java.base java.io and java.nio[.*] packages. >> >> In general, the changes consist of >> * update tables to add sc

Re: JDK 9 jar tool issues - "isolated nested class" and creation of a corrupt jar

2017-08-10 Thread Tom Hood
Glad you were able to reproduce it. Thanks for creating the JBS issue. On Thu, Aug 10, 2017 at 11:49 AM, mandy chung wrote: > > > On 8/9/17 4:50 PM, Tom Hood wrote: > >> Hi, >> >> I'm having a couple issues with the jar tool. Version: JDK 9 build 181 >> SPARC64 >> >> Since I'm typing this in, b

Re: JDK 9 jar tool issues - "isolated nested class" and creation of a corrupt jar

2017-08-10 Thread mandy chung
On 8/9/17 4:50 PM, Tom Hood wrote: Hi, I'm having a couple issues with the jar tool. Version: JDK 9 build 181 SPARC64 Since I'm typing this in, beware of typos. I cannot copy and paste output as my Solaris 10 box isn't permitted to access the internet. (1) isolated nested classes trying to