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.