On 7/10/19 6:03 PM, Joe Darcy wrote:
Hello,

I'd advise against including at least part of the comment

    // declare serialization compatibility with JDK 8 (see JDK-8227368)

We generally don't include bug numbers in the text of the code and rely mapping from the SCM to the bug database to provide the (sometimes large!) additional context for a fix.

Ok, Joe. I removed the reference to bug number and recreated webrev.03 in-place. Will that be OK?

http://cr.openjdk.java.net/~plevart/jdk-dev/8227368_EnumSet.class_serialization/webrev.03/

Regards, Peter

Reply via email to