RFR - 8132734: java.util.jar.* changes to support multi-release jar files
steve.drach at oracle.com
Thu Jan 28 19:28:59 UTC 2016
> What's the purpose of having a dedicated "JarFile.runtimeVersioned”?
Consistency, so getVersion returns the same version as the version set when JarFile was constructed.
> Based on
> its only usages at ln#356 and #381, it appears, shouldn't getVersion() simply
> returns Release.valueOf(version)?
That might be confusing. Also, having it consistent helps with testing.
> On 01/27/2016 03:37 PM, Steve Drach wrote:
>>> I'm still wondering about the phrase "root entry" as it continues to give the impression (to me anyway) that it's a resource in the root directory. I think "root" works in the JEP because it deals with simple resources like A.class and B.class that are in the root directory but it's confusing when there resources with a slash in the name. Add to this is the META-INF/versions/<n> directories which are roots for the version specific resources. I think part of the confusion is that the first mention of "root entry" is in the second paragraph where it has "overrides the unversioned root entry" without defining what it means. In summary, I'm wondering whether you would be up for change the terminology so that "root entry" isn't in the javadoc?
>> I’ve released a new webrev, http://cr.openjdk.java.net/~sdrach/8132734/webrev.04/index.html<http://cr.openjdk.java.net/~sdrach/8132734/webrev.04/index.html> that addresses the above issue.
More information about the core-libs-dev