RFR: 8202216: (bf) Add Buffer mismatch()
Peter Levart
peter.levart at gmail.com
Tue Jun 12 10:20:52 UTC 2018
Hi,
On 06/12/2018 11:34 AM, Vivek Theeyarath wrote:
> Hi All,
>
> Please review fix for https://bugs.openjdk.java.net/browse/JDK-8202216
>
>
>
> Webrev: http://cr.openjdk.java.net/~vtheeyarath/8202216/webrev.00/
>
> CSR : https://bugs.openjdk.java.net/browse/JDK-8204852
>
>
>
> Regards
>
> Vivek
>
>
This looks good as is, but would it make sense for this new method to be
defined as abstract method on Buffer? Like for example:
public abstract class Buffer<B extends Buffer> {
public abstract int mismatch(B that);
...
public abstract class ByteBuffer
extends Buffer<ByteBuffer>
implements Comparable<ByteBuffer>
{
@Override
public int mismatch(ByteBuffer that) {
...
public abstract class CharBuffer
extends Buffer<CharBuffer>
implements Comparable<CharBuffer>
{
@Override
public int mismatch(CharBuffer that) {
...
Regards, Peter
More information about the core-libs-dev
mailing list