andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

ByteBufUtil use IndexOfProcessor to find occurrence.

Browse Source 
Motivation:
The way of firstIndexOf and lastIndexOf iterating the ByteBuf is similar to forEachByte and forEachByteDesc, but have many range checks.
Modifications:
Use forEachByte and a IndexOfProcessor to find occurrence.
Result:
eliminate range checks
This commit is contained in:
Alwayswithme 2015-05-07 10:36:49 +08:00 committed by Norman Maurer
parent d98b21be04
commit c727c16707
1 changed files with 15 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
buffer/src/main/java/io/netty/buffer/ByteBufUtil.java
View File

@ -312,13 +312,7 @@ public final class ByteBufUtil {
return -1;
}
for (int i = fromIndex; i < toIndex; i ++) {
if (buffer.getByte(i) == value) {
return i;
}
}
return -1;
return buffer.forEachByte(fromIndex, toIndex - fromIndex, new IndexOfProcessor(value));
}
private static int lastIndexOf(ByteBuf buffer, int fromIndex, int toIndex, byte value) {
@ -327,13 +321,7 @@ public final class ByteBufUtil {
return -1;
}
for (int i = fromIndex - 1; i >= toIndex; i --) {
if (buffer.getByte(i) == value) {
return i;
}
}
return -1;
return buffer.forEachByteDesc(toIndex, fromIndex - toIndex, new IndexOfProcessor(value));
}
/**
@ -566,5 +554,18 @@ public final class ByteBufUtil {
}
}
private static class IndexOfProcessor implements ByteBufProcessor {
private final byte byteToFind;
public IndexOfProcessor(byte byteToFind) {
this.byteToFind = byteToFind;
}
@Override
public boolean process(byte value) {
return value != byteToFind;
}
}
private ByteBufUtil() { }
}