review request for 7005608: diamond conversion of JCA and crypto providers

Brad Wetmore bradford.wetmore at
Wed Dec 22 18:17:59 PST 2010

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 

Looks good otherwise.


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:
> (This should be substantially the same as the webrev I posted internally
> yesterday.)
> Thanks!
> s'marks
> [1]

More information about the security-dev mailing list