[ 
https://issues.apache.org/jira/browse/PDFBOX-3421?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tilman Hausherr resolved PDFBOX-3421.
-------------------------------------
       Resolution: Fixed
         Assignee: Tilman Hausherr
    Fix Version/s: 2.1.0
                   2.0.3

Setting to resolved. Thanks for the contribution!

> Optimize float to string conversion in PDAbstractContentStream
> --------------------------------------------------------------
>
>                 Key: PDFBOX-3421
>                 URL: https://issues.apache.org/jira/browse/PDFBOX-3421
>             Project: PDFBox
>          Issue Type: Improvement
>          Components: PDModel
>    Affects Versions: 2.0.2, 2.0.3, 2.1.0
>         Environment: Ubuntu 14.04
>            Reporter: Michael Doswald
>            Assignee: Tilman Hausherr
>            Priority: Trivial
>              Labels: optimization, performance
>             Fix For: 2.0.3, 2.1.0
>
>         Attachments:  PDFBOX-3421_Float_formatting_performance_rev2.patch, 
> PDFBOX-3421_Float_formatting_performance_rev1.patch, 
> PDFBOX-3421_Float_formatting_performance_rev3.patch, 
> PDFBOX-3421_Float_formatting_performance_rev4.patch, 
> pdfbox-performance-floatformat.zip
>
>
> Drawing lines in a PDF (and also other operations) writes the coordinates to 
> the content stream. Currently, the 
> PDAbstractContentStream#writeOperand(float) method uses the NumberFormat 
> class to convert the float values. This is inefficient for multiple reasons:
> * NumberFormat is designed to format numbers locale dependent. That is not 
> needed in this case
> * NumberFormat uses a pattern to format the value which is also not needed in 
> this case
> * The formatting first creates a String object, converts it to a byte array 
> with ASCII encoding and then writes it to the stream. This generates a lot of 
> garbage.
> A different approach to formatting real operands should be used.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to