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

Tilman Hausherr edited comment on PDFBOX-4909 at 7/24/20, 9:23 AM:
-------------------------------------------------------------------

Re: the image, I don't see what you mean - I see two map operations (iterator 
and computeIfAbsent) but these are called by {{processTextPosition}}. The 
changes I did are in {{showGlyph}} and there I see no map operations. I suspect 
that they are in some tiny rectangles.


was (Author: tilman):
Re: the image, I don't see what you mean - I see two map operations but these 
are called by processTextPosition. The changes I did are in showGlyph and there 
I see no map operations. I suspect that they are in some tiny rectangles.

> Don't calculate font height for every glyph
> -------------------------------------------
>
>                 Key: PDFBOX-4909
>                 URL: https://issues.apache.org/jira/browse/PDFBOX-4909
>             Project: PDFBox
>          Issue Type: Improvement
>          Components: Text extraction
>    Affects Versions: 2.0.0, 3.0.0 PDFBox
>            Reporter: Alfred
>            Assignee: Tilman Hausherr
>            Priority: Major
>              Labels: Optimization
>             Fix For: 2.0.21, 3.0.0 PDFBox
>
>         Attachments: PDFBOX-4909.patch, Untitled.png
>
>
> LegacyPDFStreamEngine computes font height for every glyph and the 
> computation is rather heavy, to work around all known problems.
> Instead of computing for every glyph, we can recompute only when the font 
> changes. The SetFontAndSize operator will be invoked when the font changes so 
> we can use that to compute and store the height to have it ready when needed 
> in showGlyph.
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to