From 92c3fd938e7a4e86edbc872eb5348dc048ec0e78 Mon Sep 17 00:00:00 2001 From: Austin Clements Date: Tue, 11 Mar 2014 18:19:54 -0400 Subject: emacs: Use whitelist instead of blacklist for term escaping Previously, the term escaper used a blacklist of characters that needed escaping. This blacklist turned out to be somewhat incomplete; for example, it did not contain non-whitespace ASCII control characters or Unicode "fancy quotes", both of which do require the term to be escaped. Switch to a whitelist of characters that are definitely safe to leave unquoted. This fixes the broken test introduced by the previous patch. --- test/T310-emacs.sh | 1 - 1 file changed, 1 deletion(-) (limited to 'test') diff --git a/test/T310-emacs.sh b/test/T310-emacs.sh index 6c18bbd7..ac966e52 100755 --- a/test/T310-emacs.sh +++ b/test/T310-emacs.sh @@ -954,7 +954,6 @@ output=$(notmuch search --output=messages 'tag:search-global-race-tag') test_expect_equal "$output" "id:$gen_msg_id_1" test_begin_subtest "Term escaping" -test_subtest_known_broken output=$(test_emacs "(mapcar 'notmuch-escape-boolean-term (list \"\" \"abc\`~\!@#\$%^&*-=_+123\" -- cgit v1.2.3