Null Pointer check in "Range" hash functions

core/src/main/java/zingg/hash/RangeDbl.java

@@ -17,7 +17,7 @@ public RangeDbl(int lower, int upper) {
 	@Override
 	public Integer call(Double field) {
 		int withinRange = 0;
-		if (field >= lowerLimit && field < upperLimit) {
+		if (field != null && field >= lowerLimit && field < upperLimit) {
 			withinRange = 1;
 		}
 		return withinRange;

core/src/main/java/zingg/hash/RangeInt.java

@@ -17,7 +17,7 @@ public RangeInt(int lower, int upper) {
 	@Override
 	public Integer call(Integer field) {
 		int withinRange = 0;
-		if (field >= lowerLimit && field < upperLimit) {
+		if (field != null && field >= lowerLimit && field < upperLimit) {
 			withinRange = 1;
 		}
 		return withinRange;

core/src/test/java/zingg/hash/TestLessThanZeroInt.java

@@ -14,8 +14,8 @@ public void testLessThanZeroIntForValueZero() {
 	}
 
 	@Test
-	public void testLessThanZeroDblForValueNull() {
-		LessThanZeroDbl value = new LessThanZeroDbl();
+	public void testLessThanZeroIntForValueNull() {
+		LessThanZeroInt value = new LessThanZeroInt();
 		assertFalse(value.call(null));
 	}
 

core/src/test/java/zingg/hash/TestRangeBetween0And10Dbl.java

@@ -48,5 +48,9 @@ public void testRangeForValue8637() {
 		RangeDbl value = new RangeBetween0And10Dbl();
 		assertEquals(0, value.call(8637d));
 	}
-
+	@Test
+	public void testRangeForNull() {
+		RangeDbl value = new RangeBetween0And10Dbl();
+		assertEquals(0, value.call(null));
+	}
 }

core/src/test/java/zingg/hash/TestRangeBetween0And10Int.java

@@ -46,5 +46,10 @@ public void testRangeForValue8637() {
 		RangeInt value = new RangeBetween0And10Int();
 		assertEquals(0, value.call(8637));
 	}
+	@Test
+	public void testRangeForNull() {
+		RangeInt value = new RangeBetween0And10Int();
+		assertEquals(0, value.call(null));
+	}
 
 }

core/src/test/java/zingg/hash/TestRangeBetween1000And10000Dbl.java

@@ -48,5 +48,10 @@ public void testRangeLessThan10000() {
 		RangeDbl value = new RangeBetween1000And10000Dbl();
 		assertEquals(1, value.call(8637d));
 	}
+	@Test
+	public void testRangeForNull() {
+		RangeDbl value = new RangeBetween1000And10000Dbl();
+		assertEquals(0, value.call(null));
+	}
 
 }

core/src/test/java/zingg/hash/TestRangeBetween1000And10000Int.java

@@ -47,5 +47,10 @@ public void testRangeForValue8637() {
 		RangeInt value = new RangeBetween1000And10000Int();
 		assertEquals(1, value.call(8637));
 	}
+	@Test
+	public void testRangeForNull() {
+		RangeInt value = new RangeBetween1000And10000Int();
+		assertEquals(0, value.call(null));
+	}
 
 }

core/src/test/java/zingg/hash/TestRangeBetween100And1000Dbl.java

@@ -49,5 +49,10 @@ public void testRangeForValue8637() {
 		RangeDbl value = new RangeBetween100And1000Dbl();
 		assertEquals(0, value.call(8637d));
 	}
+	@Test
+	public void testRangeForNull() {
+		RangeDbl value = new RangeBetween100And1000Dbl();
+		assertEquals(0, value.call(null));
+	}
 
 }

core/src/test/java/zingg/hash/TestRangeBetween100And1000Int.java

@@ -46,5 +46,10 @@ public void testRangeForValue8637() {
 		RangeInt value = new RangeBetween100And1000Int();
 		assertEquals(0, value.call(8637));
 	}
+	@Test
+	public void testRangeForNull() {
+		RangeInt value = new RangeBetween100And1000Int();
+		assertEquals(0, value.call(null));
+	}
 
 }

core/src/test/java/zingg/hash/TestRangeBetween10And100Dbl.java

@@ -49,5 +49,10 @@ public void testRangeForValue8637() {
 		RangeDbl value = new RangeBetween10And100Dbl();
 		assertEquals(0, value.call(8637d));
 	}
+	@Test
+	public void testRangeForNull() {
+		RangeDbl value = new RangeBetween10And100Dbl();
+		assertEquals(0, value.call(null));
+	}
 
 }

core/src/test/java/zingg/hash/TestRangeBetween10And100Int.java

@@ -46,5 +46,10 @@ public void testRangeForValue8637() {
 		RangeInt value = new RangeBetween10And100Int();
 		assertEquals(0, value.call(8637)); 
 	}
+	@Test
+	public void testRangeForNull() {
+		RangeInt value = new RangeBetween10And100Int();
+		assertEquals(0, value.call(null));
+	}
 
 }