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

Maruan Sahyoun commented on PDFBOX-4664:
----------------------------------------

COSArray is already implemented using a List for storage. I'm not sure if it's 
possible without breaking the API (only had a very quick look) but is we expose 
the List nature of COSArray (i.e. extend List) we could get rid of the double 
nature of COSArrayList and mainly keep the static helper methods.

> Iterator.remove() does not work in COSArrayList
> -----------------------------------------------
>
>                 Key: PDFBOX-4664
>                 URL: https://issues.apache.org/jira/browse/PDFBOX-4664
>             Project: PDFBox
>          Issue Type: Bug
>    Affects Versions: 2.0.17
>            Reporter: Zhichao Yang
>            Assignee: Andreas Lehmkühler
>            Priority: Minor
>
> {code:java}
> for (Iterator itr = pdfPage.getAnnotations().iterator(); itr.hasNext(); ) {
>                     itr.remove();
>  }
> {code}
> This is not working on annotations list because cosArrayList.iterator() is 
> cosArrayList.actual.iterator(), but when convert to a COSArray it uses 
> cosArrayList.array, which is not affected by iterator.remove(). A iterator to 
> sync the actual and the array would be nice.



--
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