[ 
https://issues.apache.org/jira/browse/PDFBOX-1646?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13872407#comment-13872407
 ] 

John Hewson commented on PDFBOX-1646:
-------------------------------------

The FOP project no longer requires the bounding box patch [1], which makes this 
issue redundant, can we close it?

[1] 
https://issues.apache.org/jira/browse/PDFBOX-1645?focusedCommentId=13871851&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13871851

> [PATCH] Add method for retrieving CFF bounding box from CFFFont class with 
> slight refactoring and optimization.
> ---------------------------------------------------------------------------------------------------------------
>
>                 Key: PDFBOX-1646
>                 URL: https://issues.apache.org/jira/browse/PDFBOX-1646
>             Project: PDFBox
>          Issue Type: Improvement
>          Components: FontBox
>    Affects Versions: 1.8.2
>            Reporter: Robert Meyer
>            Assignee: Andreas Lehmkühler
>             Fix For: 2.0.0
>
>         Attachments: patch-optimize.diff, patch.diff
>
>
> I have added a method to the CFFFont class to retrieve the bounding box for a 
> character determined by an SID as well as retrieving a name. I have also 
> slightly modified the existing code so that each sid mapping can be retrieved 
> now using the SID as the key from a map. From looking around there are 
> several examples of where iterative loops are used using the original mapping 
> array:
> CFFFontROS.java:165
> CFFParser.java:876
> I haven't changed those locations yet, but they can be made in a separate 
> patch which should boost performance.
> There was a small bit of refactoring done as well just because I now retrieve 
> a renderer from two locations. These patches are part of adding OTF CFF 
> support to Apache FOP.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to