JDK8 Code review request for 7026347: Certificate and X509CRL should have verify(PublicKey key, Provider sigProvider)

[Weijun Wang]

Hi Jason

The existing verify(PublicKey key, String sigProvider) method caches the 
result and returns it if the same key and sigProvider is used. Is it 
possible to do the same in your new method?

Also, the new method is very similar to the old one. I have no good idea 
how to reuse the old code but if you find a way that would be very nice.

As for the CCC, is it really necessary to add these lines?

   *                                           Note that the specified
   * Provider object does not have to be registered in the provider list.
   *
   * <p> This method was added to version 1.8 of the Java Platform
   * Standard Edition. In order to maintain backwards compatibility with
   * existing service providers, this method cannot be <code>abstract</code>
   * and by default throws an <code>UnsupportedOperationException</code>.

Thanks
Max

On 07/20/2012 01:03 PM, Jason Uh wrote:
> Hi all,
>
> Please review my fix for 7026347.
>
> This change is introduces verify(PublicKey key, Provider sigProvider)
> methods for Certificate and X509CRL that allow Providers not registered
> in java.security. Change has been filed with CCC and is accepted.
>
> Webrev: http://cr.openjdk.java.net/~juh/7026347/webrev.03/
> CR: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7026347
>
> Thanks,
> Jason