compare bytes to avoid allocations

we don't save the line, just the offset. using the bytes versions avoids allocating new strings Signed-off-by: Spencer Schrock <[email protected]>
ossf · Mar 7, 2024 · 904ce51 · 904ce51
1 parent 6e360b8
commit 904ce51
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/checks/raw/sast.go b/checks/raw/sast.go
@@ -314,12 +314,11 @@ func findLine(content, data []byte) (uint, error) {
 	r := bytes.NewReader(content)
 	scanner := bufio.NewScanner(r)
 
-	line := 0
-	// https://golang.org/pkg/bufio/#Scanner.Scan
+	var line uint
 	for scanner.Scan() {
 		line++
-		if strings.Contains(scanner.Text(), string(data)) {
-			return uint(line), nil
+		if bytes.Contains(scanner.Bytes(), data) {
+			return line, nil
 		}
 	}