RFR: 8072777: java/lang/ref/ReferenceEnqueuePending.java: some references aren't queued

2016-01-30 Thread Kim Barrett
Please review this fix of a bug in the ReferenceEnqueuePending test. The original code assumes the pending list processor will stay ahead of the test thread, so that the test's polling of the reference queue won't find it empty while there are still references on the internal pending list. That mig

Re: RFR: 8148626: URI.toURL needs to use protocol Handler to parse file URIs

2016-01-30 Thread Claes Redestad
Hi, On 2016-01-30 19:35, joe darcy wrote: Hello, The change looks okay in that the new code is limited to the jrt protocol. I assume the failing test java/net/URL/B5086147.java passes again with this change. I had to go through some hoops to find out how to ensure this test actually

Re: RFR: 8148626: URI.toURL needs to use protocol Handler to parse file URIs

2016-01-30 Thread joe darcy
Hello, The change looks okay in that the new code is limited to the jrt protocol. I assume the failing test java/net/URL/B5086147.java passes again with this change. If a fix for this issue is not pushed soon, I want to have the failing test problem listed (http://mail.openjdk.java.net

RFR: 8148626: URI.toURL needs to use protocol Handler to parse file URIs

2016-01-30 Thread Claes Redestad
Hi, it turns out trying to optimize URI to URL conversion for file URLs don't work out very well due to special treatment of backward slashes. Instead of trying to deal with such file URLs, this patch partially backs out 8147462 and only leave the optimization in place for the jrt protocol.

Re: RFR - 8132734: java.util.jar.* changes to support multi-release jar files

2016-01-30 Thread Alan Bateman
On 29/01/2016 17:39, Paul Sandoz wrote: : Alan’s point is that traversing using entries()/stream() always returns the versioned entries (if any) rather than all entries, thus in a sense filters. My assumption was the traversal should by default be consistent with a calls to getEntry, thus: