<Swing Dev> 7012663: javadoc warning in javax.swing.JComponent and javax.swing.plaf.synth.SynthTextPaneUI

Alexander Potochkin Alexander.Potochkin at oracle.com
Wed Jan 26 17:49:37 UTC 2011


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
>




More information about the swing-dev mailing list