RFR 8144355: JDK 9 changes to ZipFileSystem to support multi-release jar files

Xueming Shen xueming.shen at oracle.com
Tue Dec 22 01:14:39 UTC 2015


On 12/22/15, 12:41 AM, Steve Drach wrote:
>>> Hi,
>>>
>>> I updated the patch to the changeset to address Paul’s concern with a windows specific solution, as well some of Alan’s concerns.  Please let me know if this patch (instead of the previous one) can be applied to the changeset.
>>>
>>> Thanks
>>> Steve
>> Would it be possible to refresh the webrev? The patch you inlined adjusts comments and fixes the issue with the test on Windows but it's hard to see everything (like getting JarFileSystem consistent with ZipFileSystem and I think whether we should rename getEntry0 because it looks strange that this is not private).
> Sure I can refresh the webrev today.  I didn’t rename getEntry0 because that name is in ZipFileSystem and I assume it was chosen, instead of getEntry, for a reason.  Perhaps Sherman can shed some light on that.  The reason that it is not private is because JarFileSystem invokes ZipFileSystem.newInputStream, and that invokes getEntry0.
let's rename getEntry0 to getEntry(), given it's no longer a private 
method now. There
was another getEntry() in the early version of ZFS, but it does not 
matter any more.

-sherman


More information about the nio-dev mailing list