[
https://issues.apache.org/jira/browse/PDFBOX-3576?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15826316#comment-15826316
]
Tilman Hausherr commented on PDFBOX-3576:
-----------------------------------------
targetting to 2.0.5 so that we make a decision on this.
+1 from me re the PDFBOX_3576___Added_getters_equals_and_hashCode.patch which I
consider to be harmless.
> Add getter methods to TextPosition
> ----------------------------------
>
> Key: PDFBOX-3576
> URL: https://issues.apache.org/jira/browse/PDFBOX-3576
> Project: PDFBox
> Issue Type: Improvement
> Affects Versions: 2.0.3
> Reporter: Kurt Devlin
> Fix For: 2.0.5, 2.1.0
>
> Attachments: PDFBOX_3576___Added_getters_equals_and_hashCode.patch,
> PDFBOX_3576_New_getter_methods.patch
>
>
> I've inherited code in my department that created a local copy of
> TextPosition and broke it's immutability. I'm trying to refactor this code
> and use the core Apache implementation and I understand the need for
> TextPosition to be immutable, but can getters be added to provide read-only
> access to the class's fields?
> We have custom code that does comparison of endX/endY values to determine if
> there are overlapping characters and other features. In our application, we
> also have a swap feature where we need to create/clone new TextPosition
> instances with minor differences between an existing TextPosition instance.
> Since there are a lot of fields that are required by the constructor of the
> new instances, but can't be read from an existing instance, we need a way to
> access these fields.
> There probably should also be an override for equals() and hashCode().
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
