Introduce strings.TrimSuffix instead of custom split

Signed-off-by: Nikos Kakavas <nikakis@protonmail.com>
prometheus · Mar 30, 2022 · 6ff7304 · 6ff7304
1 parent 5c12586
commit 6ff7304
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/proc_netstat.go b/proc_netstat.go
@@ -192,7 +192,7 @@ func parseNetstat(r io.Reader, fileName string) (ProcNetstat, error) {
 		scanner.Scan()
 		valueParts := strings.Split(scanner.Text(), " ")
 		// Remove trailing :.
-		protocol := nameParts[0][:len(nameParts[0])-1]
+		protocol := strings.TrimSuffix(nameParts[0], ":")
 		if len(nameParts) != len(valueParts) {
 			return procNetstat, fmt.Errorf("mismatch field count mismatch in %s: %s",
 				fileName, protocol)

diff --git a/proc_snmp.go b/proc_snmp.go
@@ -157,7 +157,7 @@ func parseSnmp(r io.Reader, fileName string) (ProcSnmp, error) {
 		scanner.Scan()
 		valueParts := strings.Split(scanner.Text(), " ")
 		// Remove trailing :.
-		protocol := nameParts[0][:len(nameParts[0])-1]
+		protocol := strings.TrimSuffix(nameParts[0], ":")
 		if len(nameParts) != len(valueParts) {
 			return procSnmp, fmt.Errorf("mismatch field count mismatch in %s: %s",
 				fileName, protocol)