Hello Alan

Looks good, I approve!

Thanks
alexp
There are a couple of warnings in javadoc build that need to be squashed, including a few in the Swing area:

../../src/share/classes/javax/swing/JComponent.java:4922: warning - @true is an unknown tag. ../../src/share/classes/javax/swing/JComponent.java:4922: warning - @true is an unknown tag. ../../src/share/classes/javax/swing/plaf/synth/SynthTextPaneUI.java:99: warning - Tag @see: reference not found: BasicTextUI#installUI

These appear to be trivial to fix with the attached patch. Can someone review this and I'll get into it for the next build (probably b130)?

Thanks,

-Alan


diff -r 6e7bed89ce84 src/share/classes/javax/swing/JComponent.java
--- a/src/share/classes/javax/swing/JComponent.java Tue Jan 25 08:41:46 2011 -0500 +++ b/src/share/classes/javax/swing/JComponent.java Wed Jan 26 17:13:52 2011 +0000
@@ -4911,7 +4911,7 @@ public abstract class JComponent extends
* painting to originate from this Component, or one of its ancestors.
     * <p/>
* Calling {@link JComponent#repaint} on a Swing component will be delegated to - * the first ancestor which {@code isPaintingOrigin()} returns {@true}, + * the first ancestor which {@code isPaintingOrigin()} returns {@code true},
     * if there are any.
     * <p/>
* {@code JComponent} subclasses that need to be repainted when any of their diff -r 6e7bed89ce84 src/share/classes/javax/swing/plaf/synth/SynthTextPaneUI.java --- a/src/share/classes/javax/swing/plaf/synth/SynthTextPaneUI.java Tue Jan 25 08:41:46 2011 -0500 +++ b/src/share/classes/javax/swing/plaf/synth/SynthTextPaneUI.java Wed Jan 26 17:13:52 2011 +0000
@@ -92,7 +92,7 @@ public class SynthTextPaneUI extends Syn
     * </ol>
     *
     * @param c the editor component
-     * @see BasicTextUI#installUI
+     * @see javax.swing.plaf.basic.BasicTextUI#installUI
     * @see ComponentUI#installUI
     */
    @Override


Reply via email to