You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Matt Juntunen (Jira)" <ji...@apache.org> on 2019/12/27 19:55:00 UTC

[jira] [Commented] (GEOMETRY-65) Use of "Serializable"

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

Matt Juntunen commented on GEOMETRY-65:
---------------------------------------

Is this done? The only class I see that explicitly implements Serializable now is {{EpsilonDoublePrecisionContext}}.

> Use of "Serializable"
> ---------------------
>
>                 Key: GEOMETRY-65
>                 URL: https://issues.apache.org/jira/browse/GEOMETRY-65
>             Project: Apache Commons Geometry
>          Issue Type: Task
>          Components: core, Euclidean 1D, Euclidean 2D, Euclidean 3D, Spherical 1D, Spherical 2D
>            Reporter: Gilles Sadowski
>            Priority: Major
>              Labels: persistence, serializable
>
> Many classes are declared to be {{Serializable}}.  But in effect they are not because some of their fields are not {{Serializable}} (this is reported by "soundcloud.io").
> In my opinion, the
> {code}
> implements Serializable
> {code}
> declaration must be removed from all classes but those explicitly designed for persistence.  Actually, even for those, I'd favour a design where persistence is  implemented "externally" (i.e. *not* by {{Serializable}}).



--
This message was sent by Atlassian Jira
(v8.3.4#803005)