emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

appt.el fix: don't use prin1-to-string


From: Alan Shutko
Subject: appt.el fix: don't use prin1-to-string
Date: Mon, 12 Aug 2002 17:26:47 -0400
User-agent: Gnus/5.090007 (Oort Gnus v0.07) Emacs/21.3.50 (i686-pc-linux-gnu)

I noticed my appointment popups were looking like:

5:25pm Test" 0 11 (fontified nil)

Cause was appt.el using prin1-to-string for the entries, which had
text properties.  I don't think it shouldn't have ever been using
prin1-to-string, since those _are_ strings to begin with (and it had
code to remove the quotes that prin1-to-string added).  So they're gone.

2002-08-12  Alan Shutko  <address@hidden>

        * calendar/appt.el (appt-delete, appt-make-list): Remove
        prin1-to-string calls (and substrings around them).

--- appt.el.~1.44.~     2002-08-12 17:08:41.000000000 -0400
+++ appt.el     2002-08-12 17:14:52.000000000 -0400
@@ -454,9 +454,9 @@
   (let* ((tmp-msg-list appt-time-msg-list))
     (while tmp-msg-list
       (let* ((element (car tmp-msg-list))
-             (prompt-string (concat "Delete " 
-                                    (prin1-to-string (car (cdr element))) 
-                                    " from list? "))
+             (prompt-string (concat "Delete \"" 
+                                    (car (cdr element))
+                                    "\" from list? "))
              (test-input (y-or-n-p prompt-string)))
         (setq tmp-msg-list (cdr tmp-msg-list))
         (if test-input
@@ -512,9 +512,7 @@
              (while (and entry-list 
                          (calendar-date-equal 
                           (calendar-current-date) (car (car entry-list))))
-               (let ((time-string (substring (prin1-to-string 
-                                              (cadr (car entry-list))) 1 -1)))
-
+               (let ((time-string (cadr (car entry-list))))
                  (while (string-match
                          "\\([0-9]?[0-9]:[0-9][0-9]\\(am\\|pm\\)?\\).*"
                          time-string)


-- 
Alan Shutko <address@hidden> - In a variety of flavors!
My American Express card left home without me.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]