[cp-patches] Re: [Patch][gui] java.awt.dnd.DropTarget

[Bryce McKinlay]

Michael Koch wrote:

> Index: java/awt/dnd/DropTarget.java
> ===================================================================
> RCS file: /cvs/gcc/gcc/libjava/java/awt/dnd/DropTarget.java,v
> retrieving revision 1.7.20.1
> diff -u -r1.7.20.1 DropTarget.java
> --- java/awt/dnd/DropTarget.java        27 Sep 2004 15:14:28 -0000      1.7.20.1
> +++ java/awt/dnd/DropTarget.java        27 Jan 2005 19:13:21 -0000
> @@ -211,9 +211,10 @@
>   public void addDropTargetListener (DropTargetListener dtl)
>     throws TooManyListenersException
>   {
> -    if (dtl != null)
> -      throw new TooManyListenersException ();
> -    
> +    // Sun's JDK does not, despite documentation, throw any sort of
> +    // exception here when you install an additional DropTargetListener.
> +    // So to be compatible, we do the same thing.
> +
>     dropTargetListener = dtl;
>   }
It would be better to change the Javadoc to reflect this, rather than 
just putting in a comment.

Regards

Bryce