On Mon, 22 Jan 2024 15:38:08 GMT, Raffaello Giulietti <rgiulie...@openjdk.org> 
wrote:

>> fabioromano1 has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Removed trailing whitespaces
>
> Sure. But the purpose of a test is not much to check the _current_ code but 
> to protect against erroneous changes in the future. Without a test, a later 
> modification might introduce an undetected regression.

@rgiulietti The method `MutablebigInteger.divWord(long, int)` is used for 
division of BigIntegers, and division of BigIntegers already has several tests. 
Anyway, in case we can use the same tests which already exist for  
`Long.divideUnsigned(long, long)` and `Long.remainderUnsigned(long, long)`.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/17291#issuecomment-1904279676

Reply via email to