Looks good - although you could replace the MD5 fingerprints with the SHA256 fingerprints in the test files for some additional testing.

--Sean

On 11/23/2015 08:00 PM, Wang Weijun wrote:
Hi All

Please review a code change at

    http://cr.openjdk.java.net/~weijun/8141457/webrev.00/

SHA-256 is now the default fingerprint algorithm, and MD5 is removed in the 
full list.

No new regression test, I consider the change trivial. If you think one is 
better, I'll add it.

Thanks
Max

Reply via email to