Looks fine. Maybe you can add a comment near the definition of these constants that when a new one is added it should be listed in the isUnsupported() method.
Thanks Max > On Aug 3, 2017, at 6:18 AM, Xuelei Fan <xuelei....@oracle.com> wrote: > > Hi, > > Please review the update: > http://cr.openjdk.java.net/~xuelei/8180643/webrev.00/ > > This update is trying to detect unknown or unsupported handshake messages > sooner rather later. > > Thanks, > Xuelei
