From c6da1f64d90f0e1d4c8c7d59adc52e6862eb7ef4 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Fri, 14 Aug 2026 16:48:17 +0000 Subject: [PATCH] Simplify redundant Atoi error check in deduplicateIISNames Co-authored-by: jkroepke <1560587+jkroepke@users.noreply.github.com> --- internal/collector/iis/iis.go | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/internal/collector/iis/iis.go b/internal/collector/iis/iis.go index 49b9cb43..7f5db45a 100644 --- a/internal/collector/iis/iis.go +++ b/internal/collector/iis/iis.go @@ -322,10 +322,9 @@ func deduplicateIISNames[T collectorName](counterValues []T) []T { // Determine the numeric suffix for this entry (-1 means "no suffix"). suffix := -1 if baseName != name { - // iisCounterBaseName guarantees the remainder after '#' is numeric. - if n, err := strconv.Atoi(name[len(baseName)+1:]); err == nil { - suffix = n - } + // iisCounterBaseName guarantees the remainder after '#' is numeric, + // so the error can be safely ignored here. + suffix, _ = strconv.Atoi(name[len(baseName)+1:]) } if e, ok := seen[baseName]; ok {