Code Review Request, 8152237 Support BigInteger.TWO

Wang Weijun weijun.wang at oracle.com
Wed Mar 23 07:34:38 UTC 2016


> On Mar 23, 2016, at 12:48 PM, Xuelei Fan <Xuelei.Fan at Oracle.COM> wrote:
> 
> On 3/23/2016 12:10 PM, Wang Weijun wrote:
>> Only 3 files touched. Are you going to make the s/BigInteger.valueOf(2)/BigInteger.TWO/ changes in other files with another bug fix?
>> 
> There are also uses in security components.  I will make the update in another bug.

I see. But why is ObjectIdentifier.java included in this fix?

In you only keep BigInteger and BigDecimal, then I have no other comment.

Thanks
Max

> 
> Thanks,
> Xuelei
> 
>> Thanks
>> Max
>> 
>>> On Mar 23, 2016, at 11:26 AM, Xuelei Fan <xuelei.fan at oracle.com> wrote:
>>> 
>>> Hi,
>>> 
>>> Please review the update for the supporting of BigInteger.TWO:
>>> 
>>>   http://cr.openjdk.java.net/~xuelei/8152237/webrev/
>>> 
>>> BigInteger.valueOf(2) is a common BigInteger value used in binary and cryptography operation calculation.  The BigInteger.TWO is not exported, and hence BigInteger.valueOf(2) is used instead in applications and JDK components.  The export of static BigInteger.TWO can improve performance and simplify existing code.
>>> 
>>> Thanks,
>>> Xuelei
>> 
> 



More information about the security-dev mailing list