RFR 8199773 (bf) XXXBuffer:compareTo method is not working as expected

Paul Sandoz paul.sandoz at oracle.com
Tue Mar 20 17:45:08 UTC 2018



> On Mar 20, 2018, at 10:24 AM, Alan Bateman <Alan.Bateman at oracle.com> wrote:
> 
> 
> 
> On 20/03/2018 17:12, Paul Sandoz wrote:
>> Hi,
>> 
>> Please review this simple fix for a silly mistake when calculating the result of comparing two buffers:
>> 
>>   http://cr.openjdk.java.net/~psandoz/jdk/JDK-8199773-buffer-compare-value/webrev/ <http://cr.openjdk.java.net/~psandoz/jdk/JDK-8199773-buffer-compare-value/webrev/>
>> 
> Looks good to me, we probably should have caught this in the core review.
> 

Thanks. Hard to spot. I am glad the JCK tests found it. I should have realized the test needed to do pos/limit as well as pos/limit/slice (the latter caught bugs early in the development process).

Paul.


More information about the nio-dev mailing list