Subject: bug#10937: closed (Re: bug#10937: 24.0.94; fix savehist-printable)

From: [email-masked] (GNU bug Tracking System)
Date: Fri, 09 Mar 2012 18:14:01 +0000

Your bug report

#10937: 24.0.94; fix savehist-printable

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
10937: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=10937
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#10937: 24.0.94; fix savehist-printable
From: Leo <[email-masked]>
To: [email-masked]
Date: Sat, 10 Mar 2012 02:12:26 +0800
List-Id: <debbugs-submit.debbugs.gnu.org>

Fixed in 24.0.95.

Subject: 24.0.94; fix savehist-printable
From: Leo <[email-masked]>
To: [email-masked]
Date: Sun, 04 Mar 2012 17:15:44 +0800
List-Id: <debbugs-submit.debbugs.gnu.org>

In savehist-printable, strings are not necessary printable. They can
contain anything in their properties. Because of this I have kill-ring
emptied for almost every restart of emacs.

I propose:

diff --git a/lisp/savehist.el b/lisp/savehist.el
index c1515fa4..a3ba9f1a 100644
--- a/lisp/savehist.el
+++ b/lisp/savehist.el
@@ -369,7 +369,7 @@ (defun savehist-printable (value)
   "Return non-nil if VALUE is printable."
   (cond
    ;; Quick response for oft-encountered types known to be printable.
-   ((stringp value))
+   ((equal-including-properties value (substring-no-properties value)))
    ((numberp value))
    ((symbolp value))
    (t

or just remove (stringp value) and let the default handle it.

Leo