You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Tim Ellison (JIRA)" <ji...@apache.org> on 2009/06/20 14:02:07 UTC

[jira] Commented: (HARMONY-6239) [classlib][text]text module doesn't need to import java.awt.font while the manifest.mf file does

    [ https://issues.apache.org/jira/browse/HARMONY-6239?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12722180#action_12722180 ] 

Tim Ellison commented on HARMONY-6239:
--------------------------------------

The javadoc comment needs to be fixed too, or else we get errors about invalid references (which is why the import was added int h first place).

I agree with not putting a module dependency on AWT purely for javadoc reasons, but when we remove it we also need to remove the cause of the dependency.


> [classlib][text]text module doesn't need to import java.awt.font while the manifest.mf file does
> ------------------------------------------------------------------------------------------------
>
>                 Key: HARMONY-6239
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6239
>             Project: Harmony
>          Issue Type: Improvement
>          Components: Classlib
>    Affects Versions: 5.0M10
>         Environment: all
>            Reporter: TianYu
>            Assignee: Mark Hindess
>            Priority: Trivial
>             Fix For: 5.0M11
>
>         Attachments: diff
>
>   Original Estimate: 0.17h
>  Remaining Estimate: 0.17h
>
> I noticed that text.jar imports java.awt.font in its manifest.mf .
> After some research , I only find the following in text.jar  ( Bidi )    
>      * @see java.awt.font.TextAttribute#BIDI_EMBEDDING
>      * @see java.awt.font.TextAttribute#NUMERIC_SHAPING
>      * @see java.awt.font.TextAttribute#RUN_DIRECTION
>      */
> so I think better to delete the line

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.