From d285183e654fe412a3470e475fa1ea29e5973b78 Mon Sep 17 00:00:00 2001
From: apocelipes <seve3r@outlook.com>
Date: Mon, 18 Mar 2024 17:38:49 +0900
Subject: [PATCH] index/suffixarray: use built-in clear to simplify code

"sais2.go" is regenerated by the command "go generate ./...".
---
 src/index/suffixarray/sais.go  |  8 ++------
 src/index/suffixarray/sais2.go | 22 +++++++---------------
 2 files changed, 9 insertions(+), 21 deletions(-)

diff --git a/src/index/suffixarray/sais.go b/src/index/suffixarray/sais.go
index b53700be35..4204ba3882 100644
--- a/src/index/suffixarray/sais.go
+++ b/src/index/suffixarray/sais.go
@@ -219,9 +219,7 @@ func freq_8_32(text []byte, freq, bucket []int32) []int32 {
 	}
 
 	freq = freq[:256] // eliminate bounds check for freq[c] below
-	for i := range freq {
-		freq[i] = 0
-	}
+	clear(freq)
 	for _, c := range text {
 		freq[c]++
 	}
@@ -710,9 +708,7 @@ func recurse_32(sa, oldTmp []int32, numLMS, maxID int) {
 	// sais_32 requires that the caller arrange to clear dst,
 	// because in general the caller may know dst is
 	// freshly-allocated and already cleared. But this one is not.
-	for i := range dst {
-		dst[i] = 0
-	}
+	clear(dst)
 	sais_32(text, maxID, dst, tmp)
 }
 
diff --git a/src/index/suffixarray/sais2.go b/src/index/suffixarray/sais2.go
index 32b8972801..080c909540 100644
--- a/src/index/suffixarray/sais2.go
+++ b/src/index/suffixarray/sais2.go
@@ -14,7 +14,7 @@ func text_64(text []byte, sa []int64) {
 }
 
 func sais_8_64(text []byte, textMax int, sa, tmp []int64) {
-	if len(sa) != len(text) || len(tmp) < int(textMax) {
+	if len(sa) != len(text) || len(tmp) < textMax {
 		panic("suffixarray: misuse of sais_8_64")
 	}
 
@@ -76,7 +76,7 @@ func sais_8_64(text []byte, textMax int, sa, tmp []int64) {
 }
 
 func sais_32(text []int32, textMax int, sa, tmp []int32) {
-	if len(sa) != len(text) || len(tmp) < int(textMax) {
+	if len(sa) != len(text) || len(tmp) < textMax {
 		panic("suffixarray: misuse of sais_32")
 	}
 
@@ -138,7 +138,7 @@ func sais_32(text []int32, textMax int, sa, tmp []int32) {
 }
 
 func sais_64(text []int64, textMax int, sa, tmp []int64) {
-	if len(sa) != len(text) || len(tmp) < int(textMax) {
+	if len(sa) != len(text) || len(tmp) < textMax {
 		panic("suffixarray: misuse of sais_64")
 	}
 
@@ -208,9 +208,7 @@ func freq_8_64(text []byte, freq, bucket []int64) []int64 {
 	}
 
 	freq = freq[:256] // eliminate bounds check for freq[c] below
-	for i := range freq {
-		freq[i] = 0
-	}
+	clear(freq)
 	for _, c := range text {
 		freq[c]++
 	}
@@ -225,9 +223,7 @@ func freq_32(text []int32, freq, bucket []int32) []int32 {
 		freq = bucket
 	}
 
-	for i := range freq {
-		freq[i] = 0
-	}
+	clear(freq)
 	for _, c := range text {
 		freq[c]++
 	}
@@ -242,9 +238,7 @@ func freq_64(text []int64, freq, bucket []int64) []int64 {
 		freq = bucket
 	}
 
-	for i := range freq {
-		freq[i] = 0
-	}
+	clear(freq)
 	for _, c := range text {
 		freq[c]++
 	}
@@ -1248,9 +1242,7 @@ func recurse_64(sa, oldTmp []int64, numLMS, maxID int) {
 	// sais_64 requires that the caller arrange to clear dst,
 	// because in general the caller may know dst is
 	// freshly-allocated and already cleared. But this one is not.
-	for i := range dst {
-		dst[i] = 0
-	}
+	clear(dst)
 	sais_64(text, maxID, dst, tmp)
 }