> On Oct 19, 2016, at 1:59 PM, Steve Drach <steve.dr...@oracle.com> wrote: > >>>> In JarArchive::entries then you filter out META-INF/MANIFEST.MF and I'm >>>> not sure that that is right (think modular JAR on the module path with a >>>> manifest, it's just a resource file). >>> I took that out and it still works as expected. >>> >>>> Are directories the only case where toEntry can return null, in which case >>>> would it simpler to filter out directories here. >>> I’ve done that too. >>> >>>> There is quite a bit of clean-up needed in this area (pre-dates your patch >>>> of course). Not clear why Archive isn't a Closeable for example, or why >>>> covariant returns aren't used by the more specialized >>>> JarArchive/JarEntry/etc. I'm sure you don't want to get into that but >>>> maybe we could at least make the JarFile available via a protected method >>>> rather than a field. >>> I’ve made JarFile available via a protected method >> This sounds good. Do you have an updated patch? (the current patch is >> webrev.03 and I couldn't find a webrev.04). >> >> -Alan. > > http://cr.openjdk.java.net/~sdrach/8164805/webrev.00/ > <http://cr.openjdk.java.net/~sdrach/8164805/webrev.00/>
This webrev is not for this issue? Mandy