[debbugs-tracker] bug#10937: closed (24.0.94; fix savehist-printable)

[GNU bug Tracking System]

Your message dated Sat, 10 Mar 2012 02:12:26 +0800
with message-id <address@hidden>
and subject line Re: bug#10937: 24.0.94; fix savehist-printable
has caused the debbugs.gnu.org bug report #10937,
regarding 24.0.94; fix savehist-printable
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
10937: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=10937
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> 24.0.94; fix savehist-printable
>
>
>
>
> Date: 
>
> Sun, 04 Mar 2012 17:15:44 +0800
>
>
>
>
> 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
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#10937: 24.0.94; fix savehist-printable
>
>
>
>
> Date: 
>
> Sat, 10 Mar 2012 02:12:26 +0800
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/24.0.94 (Mac OS X 10.6.8)
>
>
>
>
> Fixed in 24.0.95.
>
>
> --- End Message ---