[ https://issues.apache.org/jira/browse/PDFBOX-4723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17009942#comment-17009942 ]
Michael Klink commented on PDFBOX-4723: --------------------------------------- Essentially {{COSArrayList}} was in need of repair. Most likely your changes have vastly improved it. Nonetheless it still is possible to shoot oneself in one's foot with it, and in some such situations one may even feel one has done nothing wrong. One thing one should test, though, is whether the addition of {{equals}} / {{hashCode}} to some COS classes makes a difference elsewhere. E.g. maps or sets of COS objects might suddenly recognize duplicates they didn't recognize before, for good or for bad... > Add equals() and hashCode() to PDAnnotation and COS objects > ----------------------------------------------------------- > > Key: PDFBOX-4723 > URL: https://issues.apache.org/jira/browse/PDFBOX-4723 > Project: PDFBox > Issue Type: Sub-task > Components: PDModel > Affects Versions: 2.0.18 > Reporter: Maruan Sahyoun > Assignee: Maruan Sahyoun > Priority: Major > Fix For: 2.0.19, 3.0.0 PDFBox > > > In order to proper support removeAll/retainAll for COSArrayList we need to > detect if entries are in fact duplicates of others. This currently fails as > even though one might add the same instance of an annotation object multiple > times to setAnnotations getting the annotations will have individual > instances. See the discussion at PDFBOX-4669. > In order to proper support removal we need to be able to detect equality > where an object is equal if the underlying COSDictionary has the same entries. -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@pdfbox.apache.org For additional commands, e-mail: dev-h...@pdfbox.apache.org