<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body text="#000000" bgcolor="#FFFFCC">
    Hi Joe,<br>
    <br>
    Looks fine to me.<br>
    <br>
    Roger<br>
    <br>
    <br>
    <div class="moz-cite-prefix">On 8/27/19 8:16 PM, Joe Darcy wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:bf56681a-c55f-be5d-8157-ba9cac793595@oracle.com">Hello,
      <br>
      <br>
      Recent work for JDK-8202385: "Annotation to mark serial-related
      fields and methods" added the java.io.Serial annotation type to
      the platform. The intention of this new annotation type is to
      allow serialization-related fields and methods to be marked as
      documentation and to allow stricter compile-time checking,
      analogous to the checking done for @Override. Implementing those
      stricter serialization-related checks will be done under
      JDK-8202056.
      <br>
      <br>
      Please review the application of java.io.Serial to the
      security-related types in the base module:
      <br>
      <br>
          JDK-8229999 : Apply java.io.Serial annotations to security
      types in java.base
      <br>
          <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~darcy/8229999.0/">http://cr.openjdk.java.net/~darcy/8229999.0/</a>
      <br>
      <br>
      As a reminder, the 5 serialization-related methods and 2 fields
      are:
      <br>
      <br>
          * private void writeObject(java.io.ObjectOutputStream stream)
      throws IOException
      <br>
          * private void readObject(java.io.ObjectInputStream stream)
      throws IOException, ClassNotFoundException
      <br>
          * private void readObjectNoData() throws ObjectStreamException
      <br>
          * ANY-ACCESS-MODIFIER Object writeReplace() throws
      ObjectStreamException
      <br>
          * ANY-ACCESS-MODIFIER Object readResolve() throws
      ObjectStreamException
      <br>
          * private static final ObjectStreamField[]
      serialPersistentFields
      <br>
          * private static final long serialVersionUID
      <br>
      <br>
      Thanks,
      <br>
      <br>
      -Joe
      <br>
      <br>
    </blockquote>
    <br>
  </body>
</html>