You need to update the Copyright updates on these files to include 2010.

Not having a lot of experience yet with <>, the only ones I wasn't sure about were the ones in X509Factor.parseX509orPKCS7Cert. (line 415 & 425) I assume it just picks up the outer return type?

Minor nit, can you add a space in line 221 between "BigInteger,BlindingParameters"

Looks good otherwise.

Brad



On 12/22/2010 3:02 PM, Stuart Marks wrote:
Hi all,

As you may have heard [1], I'm working on updating the JDK libraries to
use some Project Coin features. Currently I'm working on converting the
library code to use the "diamond" operator for more concise generic
creation expressions. Up now is conversion of the JCA and crypto
provider code. I'll be moving through other parts of the security
libraries in the JDK next.

Brad, can you review this chunk of changes? Webrev is here:

http://cr.openjdk.java.net/~smarks/reviews/7005608/webrev.0/

(This should be substantially the same as the webrev I posted internally
yesterday.)

Thanks!

s'marks

[1] http://blogs.sun.com/darcy/entry/project_coin_minty_fresh_libraries

Reply via email to