Address code review feedback - remove extra blank lines and add test

- Remove unnecessary blank lines in convertOsArgs and preprocessHelpFlags
- Add test case for -h followed by non-numeric value

Co-authored-by: dlevy-msft-sql <[email protected]>

Author: Copilot

cmd/sqlcmd/sqlcmd.go

@@ -296,7 +296,7 @@ func Execute(version string) {
 func convertOsArgs(args []string) (cargs []string) {
 	// Pre-process to handle special help flag cases
 	args = preprocessHelpFlags(args)
-	
+
 	flag := ""
 	first := true
 	for i, a := range args {
@@ -342,7 +342,7 @@ func preprocessHelpFlags(args []string) []string {
 			result = append(result, "--help")
 			continue
 		}
-		
+
 		// Handle "-h" without an argument: convert to "-?" to show help
 		if arg == "-h" {
 			// Check if next arg exists

cmd/sqlcmd/sqlcmd_test.go

@@ -78,6 +78,9 @@ func TestValidCommandLineToArgsConversion(t *testing.T) {
 		{[]string{"-help"}, func(args SQLCmdArguments) bool {
 			return args.Help
 		}},
+		{[]string{"-h", "abc"}, func(args SQLCmdArguments) bool {
+			return args.Help
+		}},
 		{[]string{"-u", "-A"}, func(args SQLCmdArguments) bool {
 			return args.UnicodeOutputFile && args.DedicatedAdminConnection
 		}},