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

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

Having taken another look at the current implementation COSArrayList doesn't 
live up to it's documentation.

{quote}
This is an implementation of a List that will sync its contents to a COSArray.
{quote}

E.g. this constructor

{code}
    public COSArrayList( List<E> actualList, COSArray cosArray )
    {
        actual = actualList;
        array = cosArray;
    }
{code}

allows to pass a List and a COSArray which is not in any way synced.



> 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