<AWT Dev> Code review request for 6909070 "Missing package statements in java.text.Bidi @see links"

[Anthony Petrov]

Looks good to me.

--
best regards,
Anthony


On 12/10/2009 06:44 AM, Joe Darcy wrote:
> Hello.
> 
> While doing a docs build, I noticed some warnings messages coming out of 
> the java.text package and I decided to try to fix them.  The problem and 
> its fix were simple: some @see links in java.text.Bidi links didn't have 
> a needed package component:
> 
> --- old/src/share/classes/java/text/Bidi.java    2009-12-09 
> 19:38:40.000000000 -0800
> +++ new/src/share/classes/java/text/Bidi.java    2009-12-09 
> 19:38:40.000000000 -0800
> @@ -121,9 +121,9 @@
>      *
>      * @param paragraph a paragraph of text with optional character and 
> paragraph attribute information
>      *
> -     * @see TextAttribute#BIDI_EMBEDDING
> -     * @see TextAttribute#NUMERIC_SHAPING
> -     * @see TextAttribute#RUN_DIRECTION
> +     * @see java.awt.font.TextAttribute#BIDI_EMBEDDING
> +     * @see java.awt.font.TextAttribute#NUMERIC_SHAPING
> +     * @see java.awt.font.TextAttribute#RUN_DIRECTION
>      */
>     public Bidi(AttributedCharacterIterator paragraph) {
>         if (paragraph == null) {
> 
> Full webrev at http://cr.openjdk.java.net/~darcy/6909070.0/
> 
> Given the simple nature of this change, I plan to commit the fix via TL.
> 
> Thanks,
> 
> -Joe