Your fix looks fine to me. Thanks.
> On 1 Feb 2016, at 22:37, Anthony Scarpino <anthony.scarp...@oracle.com> wrote: > > Hi, > > Could I have a review of the quick change that adds SHA1 and SHA224 to the > solaris-sparc preferred providers list and a minor change to handle > algorithms with two names, like SHA1 and SHA-1. > > http://cr.openjdk.java.net/~ascarpino/8145334/webrev/ > > thanks > > Tony