Re: [PATCH 1/1] Teach for-each-ref about a little language called Tcl.

From

Junio C Hamano <junkio@cox.net>

Date

Jan 28, 2007, 20:58 UTC

Message-ID

<7v4pqaeulw.fsf@assigned-by-dhcp.cox.net>

In-Reply-To

<20070128094909.GA3740@procyon.home>

Sergey Vlasov <vsu@altlinux.ru> writes:

> This is better; however, wrapping this format inside { ... } (which
> you may want to do in the template for some uses) won't work if the
> string contains unmatched braces.  Quoting '{' and '}' characters with
> backslashes should fix this.

Then I'd commit after applying this on top of Shawn's.  Ok?

diff --git a/quote.c b/quote.c
index 01db5e2..fb9e4ca 100644
--- a/quote.c
+++ b/quote.c
@@ -395,11 +395,9 @@ void tcl_quote_print(FILE *stream, const char *src)
 	fputc('"', stream);
 	while ((c = *src++)) {
 		switch (c) {
-		case '[':
-		case ']':
-		case '$':
-		case '\\':
-		case '"':
+		case '[': case ']':
+		case '{': case '}':
+		case '$': case '\\': case '"':
 			fputc('\\', stream);
 		default:
 			fputc(c, stream);