From 2ba88ccf796704272485d1f2b6e2cd85c7241127 Mon Sep 17 00:00:00 2001 From: David Levy Date: Thu, 5 Feb 2026 19:59:47 -0600 Subject: [PATCH] fix: remove redundant strings.ToLower in onerrorCommand strings.EqualFold is already case-insensitive, making the ToLower call unnecessary. --- pkg/sqlcmd/commands.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/sqlcmd/commands.go b/pkg/sqlcmd/commands.go index 66dd1dba..41250aca 100644 --- a/pkg/sqlcmd/commands.go +++ b/pkg/sqlcmd/commands.go @@ -569,9 +569,9 @@ func onerrorCommand(s *Sqlcmd, args []string, line uint) error { } params := strings.TrimSpace(args[0]) - if strings.EqualFold(strings.ToLower(params), "exit") { + if strings.EqualFold(params, "exit") { s.Connect.ExitOnError = true - } else if strings.EqualFold(strings.ToLower(params), "ignore") { + } else if strings.EqualFold(params, "ignore") { s.Connect.IgnoreError = true s.Connect.ExitOnError = false } else {