[cp-patches] Patch: PixelGrabber fix

Lillian Angel langel at redhat.com
Wed Jun 14 14:18:35 UTC 2006


On Tue, 2006-06-13 at 21:13 +0100, David Gilbert wrote:
> Lillian Angel wrote:
> 
> >On Tue, 2006-06-13 at 12:16 -0600, Tom Tromey wrote:
> >  
> >
> >>
> >>In cases like this it is nice to have a comment, either in the
> >>javadoc or in the code, explaining what happens when ip==null.
> >>That way nobody will be tempted to add the check back in later.
> >>
> >>    
> >>
> >
> >Good idea.
> >
> >2006-06-13  Lillian Angel  <langel at redhat.com>
> >
> >        * java/awt/image/PixelGrabber.java
> >        (PixelGrabber): Added to API documentation.
> >
> >
> >  
> >
> 
> And if you really want to be sure, a Mauve test will allow us all to 
> sleep well at night.  ;-)

Good idea. I have committed a test for this.

Lillian

> 
> Regards.
> 
> Dave
> 




More information about the Classpath-patches mailing list