Skip to content

Commit d2f2830

Browse files
author
Gary Gregory
committed
Remove Java 8 code.
1 parent 8b72444 commit d2f2830

2 files changed

Lines changed: 16 additions & 16 deletions

File tree

src/main/java/org/apache/commons/codec/digest/MurmurHash3.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -44,13 +44,13 @@
4444
public final class MurmurHash3 {
4545

4646
/** TODO Replace on Java 8 with Long.BYTES. */
47-
private static final int LONG_BYTES = Long.SIZE / Byte.SIZE;
47+
static final int LONG_BYTES = Long.SIZE / Byte.SIZE;
4848

4949
/** TODO Replace on Java 8 with Integer.BYTES. */
50-
private static final int INTEGER_BYTES = Integer.SIZE / Integer.SIZE;
50+
static final int INTEGER_BYTES = Integer.SIZE / Byte.SIZE;
5151

5252
/** TODO Replace on Java 8 with Short.BYTES. */
53-
private static final int SHORT_BYTES = Short.SIZE / Short.SIZE;
53+
static final int SHORT_BYTES = Short.SIZE / Byte.SIZE;
5454

5555
// from 64-bit linear congruential generator
5656
public static final long NULL_HASHCODE = 2862933555777941757L;

src/test/java/org/apache/commons/codec/digest/MurmurHash3Test.java

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -58,9 +58,9 @@ public void testHashCodeM3_64() {
5858
@Test
5959
public void test64() {
6060
final int seed = 123, iters = 1000000;
61-
ByteBuffer SHORT_BUFFER = ByteBuffer.allocate(Short.BYTES);
62-
ByteBuffer INT_BUFFER = ByteBuffer.allocate(Integer.BYTES);
63-
ByteBuffer LONG_BUFFER = ByteBuffer.allocate(Long.BYTES);
61+
ByteBuffer SHORT_BUFFER = ByteBuffer.allocate(MurmurHash3.SHORT_BYTES);
62+
ByteBuffer INT_BUFFER = ByteBuffer.allocate(MurmurHash3.INTEGER_BYTES);
63+
ByteBuffer LONG_BUFFER = ByteBuffer.allocate(MurmurHash3.LONG_BYTES);
6464
Random rdm = new Random(seed);
6565
for (int i = 0; i < iters; ++i) {
6666
long ln = rdm.nextLong();
@@ -84,7 +84,7 @@ public void test64() {
8484
@Test
8585
public void test128_Short() {
8686
// Arrange
87-
ByteBuffer BUFFER = ByteBuffer.allocate(Short.BYTES);
87+
ByteBuffer BUFFER = ByteBuffer.allocate(MurmurHash3.SHORT_BYTES);
8888
BUFFER.putShort(0, (short) 2);
8989

9090
// Act
@@ -99,7 +99,7 @@ public void test128_Short() {
9999
@Test
100100
public void test128_Int() {
101101
// Arrange
102-
ByteBuffer BUFFER = ByteBuffer.allocate(Integer.BYTES);
102+
ByteBuffer BUFFER = ByteBuffer.allocate(MurmurHash3.INTEGER_BYTES);
103103
BUFFER.putInt(0, 3);
104104

105105
// Act
@@ -114,7 +114,7 @@ public void test128_Int() {
114114
@Test
115115
public void test128_Long() {
116116
// Arrange
117-
ByteBuffer BUFFER = ByteBuffer.allocate(Long.BYTES);
117+
ByteBuffer BUFFER = ByteBuffer.allocate(MurmurHash3.LONG_BYTES);
118118
BUFFER.putLong(0, 8675309L);
119119

120120
// Act
@@ -129,7 +129,7 @@ public void test128_Long() {
129129
@Test
130130
public void test128_Double() {
131131
// Arrange
132-
ByteBuffer BUFFER = ByteBuffer.allocate(Double.BYTES);
132+
ByteBuffer BUFFER = ByteBuffer.allocate(Double.SIZE / Byte.SIZE);
133133
BUFFER.putDouble(0, 456.987);
134134

135135
// Act
@@ -179,34 +179,34 @@ public void testIncremental() {
179179

180180
@Test
181181
public void testTwoLongOrdered() {
182-
ByteBuffer buffer = ByteBuffer.allocate(Long.BYTES * 2);
182+
ByteBuffer buffer = ByteBuffer.allocate(MurmurHash3.LONG_BYTES * 2);
183183
for (long i = 0; i < 1000; i++) {
184184
for (long j = 0; j < 1000; j++) {
185185
buffer.putLong(0, i);
186-
buffer.putLong(Long.BYTES, j);
186+
buffer.putLong(MurmurHash3.LONG_BYTES, j);
187187
assertEquals(MurmurHash3.hash32(buffer.array()), MurmurHash3.hash32(i, j));
188188
}
189189
}
190190
}
191191

192192
@Test
193193
public void testTwoLongRandom() {
194-
ByteBuffer buffer = ByteBuffer.allocate(Long.BYTES * 2);
194+
ByteBuffer buffer = ByteBuffer.allocate(MurmurHash3.LONG_BYTES * 2);
195195
Random random = new Random();
196196
for (long i = 0; i < 1000; i++) {
197197
for (long j = 0; j < 1000; j++) {
198198
long x = random.nextLong();
199199
long y = random.nextLong();
200200
buffer.putLong(0, x);
201-
buffer.putLong(Long.BYTES, y);
201+
buffer.putLong(MurmurHash3.LONG_BYTES, y);
202202
assertEquals(MurmurHash3.hash32(buffer.array()), MurmurHash3.hash32(x, y));
203203
}
204204
}
205205
}
206206

207207
@Test
208208
public void testSingleLongOrdered() {
209-
ByteBuffer buffer = ByteBuffer.allocate(Long.BYTES);
209+
ByteBuffer buffer = ByteBuffer.allocate(MurmurHash3.LONG_BYTES);
210210
for (long i = 0; i < 1000; i++) {
211211
buffer.putLong(0, i);
212212
assertEquals(MurmurHash3.hash32(buffer.array()), MurmurHash3.hash32(i));
@@ -215,7 +215,7 @@ public void testSingleLongOrdered() {
215215

216216
@Test
217217
public void testSingleLongRandom() {
218-
ByteBuffer buffer = ByteBuffer.allocate(Long.BYTES);
218+
ByteBuffer buffer = ByteBuffer.allocate(MurmurHash3.LONG_BYTES);
219219
Random random = new Random();
220220
for (long i = 0; i < 1000; i++) {
221221
long x = random.nextLong();

0 commit comments

Comments
 (0)