Re: RFR: 8253100: Fix "no comment" warnings in java.base/java.net [v3]

2021-02-26 Thread Naoto Sato
On Fri, 26 Feb 2021 10:59:01 GMT, Daniel Fuchs wrote: >> Hi, >> >> Please find here a change that fixes "no comment" warnings generated by >> `javadoc -Xdoclint` for `java.base/java.net` > > Daniel Fuchs has updated the pull request incrementally with one additional > commit since the last

Re: RFR: 8262195: Harden tests that use the HostsFileNameService (jdk.net.hosts.file property) [v2]

2021-02-26 Thread Daniel Fuchs
On Thu, 25 Feb 2021 15:38:02 GMT, Conor Cleary wrote: >> A number of net tests use a >> **[HostsFileNameService](https://github.com/openjdk/jdk/blob/382e38dd246596ec94a1f1ce0e0f9e87f53366c7/src/java.base/share/classes/java/net/InetAddress.java#L955)** >> to verify either the functionality of

Integrated: JDK-8262430: doclint warnings in java.base module

2021-02-26 Thread Jonathan Gibbons
On Thu, 25 Feb 2021 22:59:23 GMT, Jonathan Gibbons wrote: > Please review some simple doc fixes in the `java.base` module. Two were > reported by doclint; the spelling error was detected by the IDE. This pull request has now been integrated. Changeset: 3088e1e6 Author:Jonathan Gibbons

Integrated: 8260366: ExtendedSocketOptions can deadlock in some circumstances

2021-02-26 Thread Jaikiran Pai
On Wed, 17 Feb 2021 07:20:27 GMT, Jaikiran Pai wrote: > Can I please get a review for this change which proposes to fix the issue > reported in https://bugs.openjdk.java.net/browse/JDK-8260366? > > The issue relates to the concurrent classloading of > `sun.net.ext.ExtendedSocketOptions` and

Re: RFR: 8260366: ExtendedSocketOptions can deadlock in some circumstances [v5]

2021-02-26 Thread Jaikiran Pai
On Wed, 24 Feb 2021 15:41:53 GMT, Daniel Fuchs wrote: >> Jaikiran Pai has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Reduce the scope of the synchronized block in getInstance() and enhance >> the testcase to be more robust in catching

Re: RFR: JDK-8262430: doclint warnings in java.base module

2021-02-26 Thread Lance Andersen
On Thu, 25 Feb 2021 22:59:23 GMT, Jonathan Gibbons wrote: > Please review some simple doc fixes in the `java.base` module. Two were > reported by doclint; the spelling error was detected by the IDE. Marked as reviewed by lancea (Reviewer). - PR:

Re: RFR: 8253100: Fix "no comment" warnings in java.base/java.net [v3]

2021-02-26 Thread Chris Hegarty
On Fri, 26 Feb 2021 10:59:01 GMT, Daniel Fuchs wrote: >> Hi, >> >> Please find here a change that fixes "no comment" warnings generated by >> `javadoc -Xdoclint` for `java.base/java.net` > > Daniel Fuchs has updated the pull request incrementally with one additional > commit since the last

Re: RFR: 8253100: Fix "no comment" warnings in java.base/java.net [v3]

2021-02-26 Thread Daniel Fuchs
> Hi, > > Please find here a change that fixes "no comment" warnings generated by > `javadoc -Xdoclint` for `java.base/java.net` Daniel Fuchs has updated the pull request incrementally with one additional commit since the last revision: Fixed @throws to conform to the style of other

Re: RFR: 8253100: Fix "no comment" warnings in java.base/java.net [v2]

2021-02-26 Thread Conor Cleary
On Fri, 26 Feb 2021 10:44:04 GMT, Rahul Yadav wrote: >> Daniel Fuchs has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Fixed comment > > LGTM! Ran the changes through `-Xdoclint:missing` and there were no warnings outputted so looks

Re: RFR: 8253100: Fix "no comment" warnings in java.base/java.net [v2]

2021-02-26 Thread Rahul Yadav
On Thu, 25 Feb 2021 11:05:07 GMT, Daniel Fuchs wrote: >> Hi, >> >> Please find here a change that fixes "no comment" warnings generated by >> `javadoc -Xdoclint` for `java.base/java.net` > > Daniel Fuchs has updated the pull request incrementally with one additional > commit since the last

Re: RFR: JDK-8262430: doclint warnings in java.base module

2021-02-26 Thread Daniel Fuchs
On Thu, 25 Feb 2021 22:59:23 GMT, Jonathan Gibbons wrote: > Please review some simple doc fixes in the `java.base` module. Two were > reported by doclint; the spelling error was detected by the IDE. LGTM - Marked as reviewed by dfuchs (Reviewer). PR:

Re: RFR: 8262195: Harden tests that use the HostsFileNameService (jdk.net.hosts.file property) [v2]

2021-02-26 Thread Chris Hegarty
On Thu, 25 Feb 2021 15:38:02 GMT, Conor Cleary wrote: >> A number of net tests use a >> **[HostsFileNameService](https://github.com/openjdk/jdk/blob/382e38dd246596ec94a1f1ce0e0f9e87f53366c7/src/java.base/share/classes/java/net/InetAddress.java#L955)** >> to verify either the functionality of

Re: RFR: JDK-8262430: doclint warnings in java.base module

2021-02-26 Thread Chris Hegarty
On Thu, 25 Feb 2021 22:59:23 GMT, Jonathan Gibbons wrote: > Please review some simple doc fixes in the `java.base` module. Two were > reported by doclint; the spelling error was detected by the IDE. Marked as reviewed by chegar (Reviewer). - PR: