Re: RFR: two minor changes to BigIntegerTest and MutableBigInteger

2013-08-02 Thread Brian Burkhalter
I filed 8022180 to track this. Thanks, Brian On Aug 1, 2013, at 4:57 PM, Brian Burkhalter wrote: > Hi Tim, > > Thanks for keeping an eye on this and for the patches. We'll need to have an > issue on file to track this. I can take care of that tomorrow. > > Brian > > On Aug 1, 2013, at 4:47

Re: RFR: two minor changes to BigIntegerTest and MutableBigInteger

2013-08-01 Thread Tim Buktu
Okay, thanks! On 02.08.2013 01:57, Brian Burkhalter wrote: > Hi Tim, > > Thanks for keeping an eye on this and for the patches. We'll need to > have an issue on file to track this. I can take care of that tomorrow. > > Brian > > On Aug 1, 2013, at 4:47 PM, Tim Buktu wrote: > >> I propose the patc

Re: RFR: two minor changes to BigIntegerTest and MutableBigInteger

2013-08-01 Thread Brian Burkhalter
Hi Tim, Thanks for keeping an eye on this and for the patches. We'll need to have an issue on file to track this. I can take care of that tomorrow. Brian On Aug 1, 2013, at 4:47 PM, Tim Buktu wrote: > I propose the patch below to fix a theoretical bug in > divideAndRemainderBurnikelZiegler().

RFR: two minor changes to BigIntegerTest and MutableBigInteger

2013-08-01 Thread Tim Buktu
I propose the patch below to fix a theoretical bug in divideAndRemainderBurnikelZiegler(). That method assumes the quotient parameter to be zero which is the case in all places it is called from. But there is no guarantee this won't change in the future and besides, the assumption is not documented