RFR: 8252996: Thread safety problem in java.net.ProxySelector

2020-09-17 Thread David M . Lloyd
8252996: Thread safety problem in java.net.ProxySelector - Commit messages: - 8252996: Fix visibility issue in ProxySelector Changes: https://git.openjdk.java.net/jdk/pull/184/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=184&range=00 Issue: https://bugs.openjdk.jav

Re: RFR: 8252996: Thread safety problem in java.net.ProxySelector

2020-09-17 Thread Alan Bateman
On Tue, 15 Sep 2020 13:54:14 GMT, David M. Lloyd wrote: > 8252996: Thread safety problem in java.net.ProxySelector Seems like an oversight when proxy selector was added - Marked as reviewed by alanb (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/184

Re: RFR: 8252996: Thread safety problem in java.net.ProxySelector

2020-09-17 Thread Daniel Fuchs
On Thu, 17 Sep 2020 12:07:06 GMT, Alan Bateman wrote: >> 8252996: Thread safety problem in java.net.ProxySelector > > Seems like an oversight when proxy selector was added Hi David, I can sponsor this for you - unless you already have a sponsor? best regards, -- daniel - PR: http

Re: RFR: 8252996: Thread safety problem in java.net.ProxySelector

2020-09-17 Thread David M . Lloyd
On Thu, 17 Sep 2020 13:04:28 GMT, Daniel Fuchs wrote: >> Seems like an oversight when proxy selector was added > > Hi David, > > I can sponsor this for you - unless you already have a sponsor? > > best regards, > -- daniel No, I do not (though @ChrisHegarty did take a peek at my original ML po

Integrated: 8252996: Thread safety problem in java.net.ProxySelector

2020-09-17 Thread David M . Lloyd
On Tue, 15 Sep 2020 13:54:14 GMT, David M. Lloyd wrote: > 8252996: Thread safety problem in java.net.ProxySelector This pull request has now been integrated. Changeset: cca3a26e Author:David M. Lloyd Committer: Daniel Fuchs URL: https://git.openjdk.java.net/jdk/commit/cca3a26e Stat

Re: RFR: 8252996: Thread safety problem in java.net.ProxySelector

2020-09-17 Thread Daniel Fuchs
On Thu, 17 Sep 2020 13:16:41 GMT, David M. Lloyd wrote: >> Hi David, >> >> I can sponsor this for you - unless you already have a sponsor? >> >> best regards, >> -- daniel > > No, I do not (though @ChrisHegarty did take a peek at my original ML post). Done. - PR: https://git.ope

Re: RFR: 8252999: Cleanup: replace .equals("") with .isEmpty() within all codebase

2020-09-17 Thread Stuart Marks
On Fri, 11 Sep 2020 15:17:58 GMT, Bradford Wetmore wrote: >> Ok, sorry for the distraction. > > Our local Santuario maintainer says: > > In general, changes to Apache Santuario should also be made at Apache so we > stay in sync. Hi @doom369, I hope we didn't end up wasting too much of your tim