Am Dienstag, den 18.10.2005, 22:58 +0200 schrieb Robert Schuster:
> Hi Roman,
>
> > Index: javax/swing/colorchooser/DefaultSwatchChooserPanel.java
> > ===================================================================
> > RCS file:
> > /cvsroot/classpath/classpath/javax/swing/colorchooser/DefaultSwatchChooserPanel.java,v
> > retrieving revision 1.5
> > diff -u -r1.5 DefaultSwatchChooserPanel.java
> > --- javax/swing/colorchooser/DefaultSwatchChooserPanel.java 2 Jul 2005
> > 20:32:49 -0000 1.5
> > +++ javax/swing/colorchooser/DefaultSwatchChooserPanel.java 18 Oct 2005
> > 20:28:28 -0000
> > @@ -587,6 +587,7 @@
> > */
> > public void addLayoutComponent(String name, Component comp)
> > {
> > + // Nothing to do here.
> > }
> >
> > /**
> > @@ -634,6 +635,7 @@
> > */
> > public void removeLayoutComponent(Component comp)
> > {
> > + // Nothing to do here.
> > }
> >
> > /**
> > @@ -786,6 +788,7 @@
> > */
> > public void updateChooser()
> > {
> > + // Nothing to do here yet.
> > }
> >
> I dont understand this. Does this mean
>
> // TODO: implement this
>
> or not?No, it doesn't. In this particular case, this method has to be overridden, since it is declared abstract in the superclass, but there is no use of it in this place. At least, I cannot see any. This is why I put in // Nothing to do here. and not a // TODO. /Roman
signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil
_______________________________________________ Classpath-patches mailing list Classpath-patches@gnu.org http://lists.gnu.org/mailman/listinfo/classpath-patches
