On Fri, 2006-07-14 at 10:10 -0400, Tania Bento wrote:
> Hey,
> 
> According to Java's API, setHgap and setVgap do not throw an Illegal
> Argument Exception if hgap or vgap are negative values.  
> 
> Two Intel tests now pass on Classpath
> (test.java.awt.FlowLayoutTest.testGetSetHgap() and
> test.java.awt.FlowLayoutTest.testGetSetVgap()).
> 
> Could someone please comment on and/or approve this patch.

Can you update the comments for these functions to reflect the changes?
Otherwise, someone is bound to put the exception throwing back in. 

I think it is good fine to commit.

Lillian

> 
> Thanks.
> Tania
> 
> 2006-07-14  Tania Bento  <[EMAIL PROTECTED]>
>       
>       * java/awt/FlowLayout.java
>       (setHgap): Illegal Argument Exception should not be 
>       thrown if hgap has a negative value.
>       (setVgap): Illegal Argument Exception should not be
>       thrown if vgap has a negative value.


Reply via email to