Re: bug in java.util.Map

[Eric Blake]

That is not a bug. Map is an interface, so all inner types are implicitly 
public.  Our coding standard discourages redundant modifiers, so please do not 
apply this patch.

Matthias Pfisterer wrote:
> Hi,
>
> I found a minor bug in java.util.Map: the inner class Entry has to be 
> public. Diff is attached.
>
> Matthias
>
>
> ------------------------------------------------------------------------
>
> Index: java/util/Map.java
> ===================================================================
> RCS file: /cvsroot/classpath/classpath/java/util/Map.java,v
> retrieving revision 1.11
> diff -u -r1.11 Map.java
> --- java/util/Map.java  15 Oct 2003 12:29:35 -0000      1.11
> +++ java/util/Map.java  20 Jan 2004 20:34:32 -0000
> @@ -264,7 +264,7 @@
>     * @since 1.2
>     * @status updated to 1.4
>     */
> -  static interface Entry
> +  public static interface Entry
>    {
>      /**
>       * Get the key corresponding to this entry.
>
>
> ------------------------------------------------------------------------
>
> _______________________________________________
> Classpath mailing list
> address@hidden
> http://mail.gnu.org/mailman/listinfo/classpath

--
Someday, I might put a cute statement here.

Eric Blake             address@hidden