<html>

  <head>

    <meta content="text/html; charset=ISO-8859-1"

      http-equiv="Content-Type">

  </head>

  <body text="#000000" bgcolor="#FFFFFF">

    <div class="moz-cite-prefix">On 3/20/15 7:39 AM, Wang Weijun wrote:<br>

    </div>

    <blockquote

      cite="mid:83DE38C5-ED13-46A4-A75E-7D8748B2EFC9@oracle.com"

      type="cite">

      <pre wrap="">Please review the code changes at

   <a moz-do-not-send="true" class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/%7Eweijun/8047789/webrev.03/">http://cr.openjdk.java.net/~weijun/8047789/webrev.03/</a></pre>

    </blockquote>

    <br>

    <pre><meta http-equiv="content-type" content="text/html; charset=ISO-8859-1">It looks okay to me.  Removing some use of reflection seems 

a good thing.

One observation: invoke(String methodName) now calls the method

statically and the implementation has conditional code depending

on which of the four methods.  It seems that you could do some

refactoring to define 4 different methods for example:

 571        invokePriv(LOGIN_METHOD) -> invokeLoginMethod

This is an separate issue.  Just to mention for future cleanup 

to consider.

Mandy

<pre></pre><pre></pre></pre>

  </body>

</html>