<html>

  <head>

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

      http-equiv="Content-Type">

  </head>

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

    <pre>Vinnie,

I am still reviewing the changes.

Just some quick comments and questions:

Your current changes have both PBES1 and PBES2 impl inside the PBECipherCore.java but yet there is 

another new class <font color="#000000">PBES2Core.java</font> which seems to just delegate the calls to the internal PBECipherCore

object.

Have you considered splitting out the PBES2 impl from PBECipherCore into PBES2Core.java?

Maybe renaming PBECipherCore to PBES1Core.java so it's clear that it's PBES1 impl only.

Also, instead of encapsulating the PBEXXCore object and yet duplicating all the engineXXX methods 

which simply delegate to the implXXX calls, why not just extending from a common parent which extends

the CipherSpi class and no need for the engineXXX methods for all children classes.

Thanks,

Valerie

</pre>

    <br>

    On 06/01/12 11:18, Vincent Ryan wrote:

    <blockquote cite="mid:4FC90790.5070109@oracle.com" type="cite">Hello

      Valerie,

      <br>

      <br>

      Could you please review these changes for JEP-121:

      <br>

        <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~vinnie/6383200/webrev.00/">http://cr.openjdk.java.net/~vinnie/6383200/webrev.00/</a>

      <br>

      <br>

      Thanks.

      <br>

    </blockquote>

    <br>

  </body>

</html>