Re: [cp-patches] FYI: FlowLayout fixlet

[Roman Kennke]

> > Roman>    public void setAlignment (int align)
> > Roman>    {
> > Roman> -    if (align != LEFT && align != RIGHT && align != CENTER
> > Roman> -    && align != LEADING && align != TRAILING)
> > Roman> -      throw new IllegalArgumentException ("invalid alignment: " + 
> > align);
> > Roman>      this.align = align;
> > Roman>    }
> >  
> > It would be nice to have a comment here explaining why we don't do
> > the check.
> 
> Ok, I'll add this.

Added.

2005-08-18  Roman Kennke  <address@hidden>

        * java/awt/FlowLayout.java
        (setAlignment): Added comment explaining why we don't check
        for illegal values here.

/Roman

FlowLayout.diff
Description: Text Data