RFR: JDK-8192935 Fix EnumSet's SerializationProxy javadoc

Roger Riggs Roger.Riggs at Oracle.com
Mon Dec 4 20:38:54 UTC 2017


Keep the comment, I didn't notice it had only been relocated.

Thanks,

On 12/4/2017 3:36 PM, Martin Buchholz wrote:
> Hi Roger,
>
> On Mon, Dec 4, 2017 at 12:12 PM, Roger Riggs <Roger.Riggs at oracle.com 
> <mailto:Roger.Riggs at oracle.com>> wrote:
>
>     Hi Martin,
>
>     The java.time APIs refined the pattern used for Serialization
>     proxies to document the relationship between
>     the original class and its serialization proxies methods.
>
>
> Right.  I was aware of the effort that java.time people put into their 
> serialization code.  They did a good job, and my proposed change makes 
> it more like theirs (but even more like the ones in j.u.c.a.)
:)
>
>     452: SerializationProxy.readResolve doesn't need the comment about
>     elementType.cast
>     since EnumSet.add method checks each element that is added.
>
>
> You may be right, but I was only preserving the original comment, and 
> removing it should be a separate change (I leave it to you!).



More information about the core-libs-dev mailing list