added comments and got rid of fixed variable

surgebuild · Apr 6, 2018 · 4bcfcd5 · 4bcfcd5
1 parent 348e28a
commit 4bcfcd5
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/Gopkg.lock b/Gopkg.lock
diff --git a/types/store.go b/types/store.go
@@ -231,25 +231,26 @@ func (key *KVStoreKey) String() string {
 	return fmt.Sprintf("KVStoreKey{%p, %s}", key, key.name)
 }
 
-// TODO: Move to TmLibs
+// PrefixEndBytes returns the []byte that would end a
+// range query for all []byte with a certain prefix
+// Deals with last byte of prefix being FF without overflowing
 func PrefixEndBytes(prefix []byte) []byte {
 	if prefix == nil {
 		return nil
 	}
 
 	end := make([]byte, len(prefix))
 	copy(end, prefix)
-	finished := false
 
-	for !finished {
+	for {
 		if end[len(end)-1] != byte(255) {
 			end[len(end)-1]++
-			finished = true
+			break
 		} else {
 			end = end[:len(end)-1]
 			if len(end) == 0 {
 				end = nil
-				finished = true
+				break
 			}
 		}
 	}