classpath
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Patch: image loading fixes


From: Tom Fitzsimmons
Subject: Re: Patch: image loading fixes
Date: Wed, 12 Nov 2003 11:44:26 -0500

On Wed, 2003-11-12 at 03:12, Sascha Brawer wrote:
> Hi Tom,
> 
> Thomas Fitzsimmons <address@hidden> wrote on Tue, 11 Nov
> 2003 16:33:30 -0500:
> 
> >[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.

Thanks,
Tom






reply via email to

[Prev in Thread] Current Thread [Next in Thread]