Subject: Re: bug#25150: [PATCH] Removing ] from message-cite-prefix-regexp

From: nitish chandra <[email-masked]>
Date: Tue, 20 Dec 2016 10:34:40 +0530

On 20 Dec 2016 1:14 am, "Eli Zaretskii" <[email-masked]> wrote:
>
> > From: nitish chandra <[email-masked]>
> > Date: Tue, 20 Dec 2016 00:52:27 +0530
> >
> > I've a question about the code though. Why is the same variable defined
in both lisp/ldefs-boot.el and
> > lisp/mail/sendmail.el? I changed the default variable value in both the
places.
>
> Auto-loaded symbols appear in loaddefs.el as well.  ldefs-boot.el is
> the version of loaddefs.el needed for bootstrapping Emacs (loaddefs.el
> is a generated file, but you need Emacs to generate it).

Oh. Cool. Then the code should be changed only in sendmail.el. Following is
the new patch.

Thanks
Nitish
From 1de7da4495f2e544cf100515d9c052808d115815 Mon Sep 17 00:00:00 2001
From: nitishch <[email-masked]>
Date: Tue, 20 Dec 2016 10:17:42 +0530
Subject: [PATCH] Fix Bug#25150

---
 lisp/mail/sendmail.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lisp/mail/sendmail.el b/lisp/mail/sendmail.el
index 3d22209..b8cf632 100644
--- a/lisp/mail/sendmail.el
+++ b/lisp/mail/sendmail.el
@@ -273,7 +273,7 @@ mail-citation-header
 
 ;;;###autoload
 (defcustom mail-citation-prefix-regexp
-  (purecopy "\\([ \t]*\\(\\w\\|[_.]\\)+>+\\|[ \t]*[]>|]\\)+")
+  (purecopy "\\([ \t]*\\(\\w\\|[_.]\\)+>+\\|[ \t]*[>|]\\)+")
   "Regular expression to match a citation prefix plus whitespace.
 It should match whatever sort of citation prefixes you want to handle,
 with whitespace before and after; it should also match just whitespace.
-- 
2.9.0