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