fix: clarify empty-string-to-0 default scope in getOptionalIntArgument

Document that the empty-value path only applies to -r (errorsToStderr)
bare flag usage; other callers always supply a value.

Author: dlevy-msft-sql

cmd/sqlcmd/sqlcmd.go

@@ -376,7 +376,8 @@ func getOptionalIntArgument(cmd *cobra.Command, name string) (i *int) {
 	if val != nil && val.Changed {
 		i = new(int)
 		value := val.Value.String()
-		// Handle empty value for flags that allow no argument (e.g., -r without value defaults to 0)
+		// Bare flag with no argument (e.g., -r) defaults to 0.
+		// This only applies to -r (errorsToStderr); other callers always provide a value.
 		if value == "" {
 			*i = 0
 			return