Thread (1 message) 1 message, 1 author, 2026-02-24

Re: [PATCH] send-email: validate charset name in 8bit encoding prompt

From: Junio C Hamano <hidden>
Date: 2026-02-24 21:11:07

Shreyansh Paliwal [off-list ref] writes:
+			if ($warn_invalid) {
+				if (find_encoding($resp))
+					return $resp;
+				else
+					printf STDERR __("warning: '%s' does not appear to be a valid charset name.\n"), $resp;
+			} else
+				return $resp;
This is not C but Perl.

 git-send-email.perl | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)
diff --git i/git-send-email.perl w/git-send-email.perl
index dc4e5418d3..15387ac377 100755
--- i/git-send-email.perl
+++ w/git-send-email.perl
@@ -1008,12 +1008,14 @@ sub ask {
 		}
 		if (!defined $valid_re or $resp =~ /$valid_re/) {
 			if ($warn_invalid) {
-				if (find_encoding($resp))
+				if (find_encoding($resp)) {
 					return $resp;
-				else
+				} else {
 					printf STDERR __("warning: '%s' does not appear to be a valid charset name.\n"), $resp;
-			} else
+				}
+			} else {
 				return $resp;
+			}
 		}
 		if ($confirm_only) {
 			my $yesno = $term->readline(
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help