Thanks Tom. I've just committed it.
On Tue, 2006-06-13 at 12:07 -0400, Thomas Fitzsimmons wrote:
> Tania Bento wrote:
> > Hey,
> >
> > Before setting the value of the button's selected property to true, i
> > added a check that ensures that state first be set to true.
> >
> > 2006-06-13 Tania Bento <[EMAIL PROTECTED]>
> >
> > * classpath/javax/swing/JCheckBoxMenuItem.java
> > (JCheckBoxMenuItem): Added check to set the selected state.
> >
> >
> >
> >
> > ------------------------------------------------------------------------
> >
> > Index: JCheckBoxMenuItem.java
> > ===================================================================
> > RCS file: /cvsroot/classpath/classpath/javax/swing/JCheckBoxMenuItem.java,v
> > retrieving revision 1.16
> > diff -u -r1.16 JCheckBoxMenuItem.java
> > --- JCheckBoxMenuItem.java 14 May 2006 21:00:28 -0000 1.16
> > +++ JCheckBoxMenuItem.java 13 Jun 2006 15:20:24 -0000
> > @@ -1,39 +1,39 @@
> > /* JCheckBoxMenuItem.java --
> >
> > /**
> > - * Creates a new JCheckBoxMenuItem object with given label, icon,
> > - * and marked as checked if given 'state' is true.
> > - *
> > + * Creates a new JCheckBoxMenuItem object with given label, icon, and
> > marked
> > + * as checked if given 'state' is true.
> > + *
> > * @param text Label for this menu item
> > * @param icon icon for this menu item
> > - * @param state <code>true</code> if this item should be in checked
> > state and
> > - * false otherwise
> > + * @param state <code>true</code> if this item should be in checked state
> > + * and false otherwise
> > */
> > public JCheckBoxMenuItem(String text, Icon icon, boolean state)
> > {
> > @@ -148,12 +148,14 @@
> > setModel(new JToggleButton.ToggleButtonModel());
> > this.state = state;
> > this.setVisible(true);
> > + if (state == true)
> > + this.setSelected(true);
> > }
>
> Approved.
>
> Tom
