RFR: JDK-6996366 : convert MacAlg to an enum
Jamil Nimeh
jamil.j.nimeh at oracle.com
Sat Mar 14 10:54:52 UTC 2015
Or we could just take care of it now, since I'm already making enums out
of things.
http://cr.openjdk.java.net/~jnimeh/reviews/6996366/webrev.02/
I don't see any differences in jtreg results across the 211 ssl tests I
ran so that seems like a good sign. I've got a jprt run going now to
see if anything happens there. Let me know what you think.
--Jamil
On 03/14/2015 01:08 AM, Xuelei Fan wrote:
> Looks fine to me.
>
> Do you want to consider a similar conversion on BulkCipher? Maybe in a
> new bug.
>
> Thanks,
> Xuelei
>
> On 3/12/2015 7:55 AM, Jamil Nimeh wrote:
>> Hello all,
>>
>> This bug moves the internal MacAlg concrete class to an enum, and alters
>> the CipherSuite constructor to no longer use String parsing on the
>> cipher suite name to determine the MacAlg. Instead, the constructor now
>> requires the caller to pass in a MacAlg, similar to how it already takes
>> a KeyExchange and BulkCipher.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-6996366
>> Webrev: http://cr.openjdk.java.net/~jnimeh/reviews/6996366/webrev.01/
>>
>> Thanks,
>> --Jamil
More information about the security-dev
mailing list