Surely the exception to throw here would be IllegalArgumentException
anyway, as null would be an illegal input to the method.
Ricky.
> > >[patch to java/awt/Component.java]
> >
> > public boolean prepareImage(Image image, ImageObserver observer)
> > {
> > + if (image == null)
> > + throw new NullPointerException ();
> > +
> > return prepareImage(image, image.getWidth(observer),
> > image.getHeight(observer), observer);
> > }
> >
> > Actually, the test for image == null is not needed.
> >
>
> Yes, I know an NPE will be thrown either way. I just thought it better
> to be explicit. But the other methods in Component don't follow this
> policy so I'll remove the check.
--
Phasers locked on target, Captain.
_______________________________________________
Classpath mailing list
[EMAIL PROTECTED]
http://mail.gnu.org/mailman/listinfo/classpath