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

Tilman Hausherr commented on PDFBOX-6085:
-----------------------------------------

It turns out that this bug was introduced 7 months ago in the related issue, 
with the line {{invisibleGlyphIds.contains(gid)}}. This line should have been 
in {{addCompoundReferences()}} as well. Changing this fixes the problem. I'll 
fix this and also make some cosmetic changes, and then research if this segment 
breaks the positioning. (We don't use seek(), we keep track on the current 
position and then skip forward accordingly and I wonder if that code segment 
does this properly)

> TTFSubsetter.buildGlyfTable() modifies glyphIds while iterating over its 
> entries possibly causing ConcurrentModificationException to be thrown
> ----------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: PDFBOX-6085
>                 URL: https://issues.apache.org/jira/browse/PDFBOX-6085
>             Project: PDFBox
>          Issue Type: Bug
>    Affects Versions: 2.0.35, 3.0.6 PDFBox
>            Reporter: Constantine Dokolas
>            Priority: Major
>
> Modifying a {{TreeSet}} while iterating over its members is prohibited as it 
> may throw a {{ConcurrentModificationException}} if the set is actually 
> modified.
> This can be observed during subsetting some fonts, e.g. Devanagari 
> (NotoSansDevanagari).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

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

Reply via email to