Here’s a new webrev that addresses Claes’ and Paul’s concerns http://cr.openjdk.java.net/~sdrach/8163798/webrev.03/ <http://cr.openjdk.java.net/~sdrach/8163798/webrev.03/>
> On Sep 11, 2016, at 1:12 PM, Steve Drach <steve.dr...@oracle.com> wrote: > > I made a simple change, the new webrev is > http://cr.openjdk.java.net/~sdrach/8163798/webrev.02/ > <http://cr.openjdk.java.net/~sdrach/8163798/webrev.02/> > >> On Sep 9, 2016, at 4:02 PM, Steve Drach <steve.dr...@oracle.com> wrote: >> >> Hi, >> >> Please review this changeset that adds a VersionedStream class to the >> jdk.internal.util.jar package. Some may recall that I submitted a similar >> RFR a few weeks ago; this is a redesign from that one. We decided not to >> make a public JarFile::versionedStream method at this time. Once we get >> sufficient experience with this and find a few more use cases, we will >> revisit the idea of making this a public method in JarFile. >> >> issue: https://bugs.openjdk.java.net/browse/JDK-8163798 >> <https://bugs.openjdk.java.net/browse/JDK-8163798> >> webrev: http://cr.openjdk.java.net/~sdrach/8163798/webrev.01/index.html >> <http://cr.openjdk.java.net/~sdrach/8163798/webrev.01/index.html> >> >> Thanks, >> Steve >
