Re: [13] RFR: 8214565 and 8214563: Use {@systemProperty} for definitions of system properties

2019-05-22 Thread Roger Riggs
PM To: core-libs-dev@openjdk.java.net Subject: Re: [13] RFR: 8214565 and 8214563: Use {@systemProperty} for definitions of system properties Hi, AsynchronousChannelProvider.java: line 144:  needs a space in "anunspecified"   That sentence isn't very well worded, but is outside th

RE: [13] RFR: 8214565 and 8214563: Use {@systemProperty} for definitions of system properties

2019-05-22 Thread Deepak Kejriwal
: 8214565 and 8214563: Use {@systemProperty} for definitions of system properties Hi, AsynchronousChannelProvider.java: line 144:  needs a space in "anunspecified"   That sentence isn't very well worded, but is outside the scope of this change to add the tag. Otherwise, looks fine

Re: [13] RFR: 8214565 and 8214563: Use {@systemProperty} for definitions of system properties

2019-05-17 Thread Roger Riggs
Hi, AsynchronousChannelProvider.java: line 144:  needs a space in "anunspecified"   That sentence isn't very well worded, but is outside the scope of this change to add the tag. Otherwise, looks fine. Roger On 05/17/2019 06:49 AM, Alan Bateman wrote: On 17/05/2019 10:49, Deepak Kejriwal

Re: [13] RFR: 8214565 and 8214563: Use {@systemProperty} for definitions of system properties

2019-05-17 Thread Alan Bateman
On 17/05/2019 10:49, Deepak Kejriwal wrote: Hi all, Please review the fix for following issues:- https://bugs.openjdk.java.net/browse/JDK-8214565 https://bugs.openjdk.java.net/browse/JDK-8214563 Below is the webrev for above issues: http://cr.openjdk.java.net/~dkejriwal/8

[13] RFR: 8214565 and 8214563: Use {@systemProperty} for definitions of system properties

2019-05-17 Thread Deepak Kejriwal
Hi all, Please review the fix for following issues:- https://bugs.openjdk.java.net/browse/JDK-8214565 https://bugs.openjdk.java.net/browse/JDK-8214563 Below is the webrev for above issues: http://cr.openjdk.java.net/~dkejriwal/8214565_8214563/webrev.00/ Regards, Deepak