Re: [j-devel] PATCH: Bignum exponent for EXPT

[Robert D. <rob...@ya...>]

--- On Wed, 8/13/08, XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX wrote:

> I was reading your patch and I saw you're doing:
> 
>  y = y.divideBy(BIGINTEGER_TWO);
> 
> Wouldn't it be better to use:
> 
>  y = y.shiftLeft(1);

I guess so, if y is positive. (I think there is some logic
to ensure that y is positive, but I don't remember exactly.)

Thanks for your help,

Robert Dodier