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