On 22 Dec 2015, at 14:53, Aleksey Shipilev <[email protected]> wrote:
> On 12/22/2015 05:41 PM, Chris Hegarty wrote: >> sun.misc.CompoundEnumeration is a utility class that supports enumerating >> over an array of enumerations. It is used in just a single place in the JDK, >> in >> ClassLoader.getResources(String) to support enumerating over the parent’s >> resources as well as the current loaders resources. I can find no usages >> outside of the JDK. >> >> This issue proposes to move sun.mics.CompoundEnumeration and its only >> use-site, in ClassLoader. >> >> http://cr.openjdk.java.net/~chegar/8146000/ > > Looks good. Let me have my share of bike-shedding: > > * Class should be final > * $enums field should be final > * $index initialization can be dropped Thanks Aleksey, All comments accepted and done. Updated webrev in-place: http://cr.openjdk.java.net/~chegar/8146000/ -Chris.
