Correct upper bound in for loop

tedyu · tedyu · commit 1719c5b0d820 · 2015-05-06T13:43:47.000-07:00
diff --git a/unsafe/src/main/java/org/apache/spark/unsafe/bitset/BitSetMethods.java b/unsafe/src/main/java/org/apache/spark/unsafe/bitset/BitSetMethods.java
@@ -74,7 +74,7 @@ public static boolean anySet(Object baseObject, long baseOffset, long bitSetWidt
     assert bitSetWidthInBytes % WORD_SIZE == 0;
     int widthInLong = (int)(bitSetWidthInBytes / WORD_SIZE);
     long addr = baseOffset;
-    for (int i = 0; i <= widthInLong; i++, addr += WORD_SIZE) {
+    for (int i = 0; i < widthInLong; i++, addr += WORD_SIZE) {
       if (PlatformDependent.UNSAFE.getLong(baseObject, addr) != 0) {
         return true;
       }

Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ public static boolean anySet(Object baseObject, long baseOffset, long bitSetWidt`
`74`	`74`	`assert bitSetWidthInBytes % WORD_SIZE == 0;`
`75`	`75`	`int widthInLong = (int)(bitSetWidthInBytes / WORD_SIZE);`
`76`	`76`	`long addr = baseOffset;`
`77`		`- for (int i = 0; i <= widthInLong; i++, addr += WORD_SIZE) {`
	`77`	`+ for (int i = 0; i < widthInLong; i++, addr += WORD_SIZE) {`
`78`	`78`	`if (PlatformDependent.UNSAFE.getLong(baseObject, addr) != 0) {`
`79`	`79`	`return true;`
`80`	`80`	`}`