Quick code review request for 634992 (enum) Include links from java.lang.Enum to EnumSet and EnumMap

[Neal Gafter]

Generally speaking, lower levels in the system shouldn't refer to higher
levels.  In this case it's just a comment, but it isn't clear that every
profile that includes Enum will include the collections framework.  I'm
thinking, for example, about ME profiles.

-Neal

On Fri, Oct 9, 2009 at 4:13 PM, Joe Darcy <Joe.Darcy at sun.com> wrote:

> Neal Gafter wrote:
>
>> Do EnumSet and EnumMap exist on all platform profiles?
>>
>
> They exist on JDK 7 where this change is intended.
>
> -Joe
>
>
>  On Fri, Oct 9, 2009 at 11:43 AM, Joseph D. Darcy <Joe.Darcy at sun.com>
>> wrote:
>>
>>> Please review this tiny doc fix to make EnumSet and EnumMap just a bit
>>> easier to find; patch below
>>>
>>> -Joe
>>>
>>> --- old/src/share/classes/java/lang/Enum.java    2009-10-09
>>> 11:41:23.000000000 -0700
>>> +++ new/src/share/classes/java/lang/Enum.java    2009-10-09
>>> 11:41:23.000000000 -0700
>>> @@ -44,6 +44,8 @@
>>>  * @author  Josh Bloch
>>>  * @author  Neal Gafter
>>>  * @see     Class#getEnumConstants()
>>> + * @see     java.util.EnumSet
>>> + * @see     java.util.EnumMap
>>>  * @since   1.5
>>>  */
>>> public abstract class Enum<E extends Enum<E>>
>>>
>>>
>>>
>>>
>>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/core-libs-dev/attachments/20091009/4154eb2a/attachment.html>