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