]> code.delx.au - gnu-emacs/blobdiff - lisp/calendar/appt.el
*** empty log message ***
[gnu-emacs] / lisp / calendar / appt.el
index e36a9ece197ff999dc89c461e7eb169a6fd7eb9c..8c8076a841364d76354e9cde002afe463ec94f4f 100644 (file)
@@ -1,9 +1,9 @@
 ;;; appt.el --- appointment notification functions.
 
-;; Copyright (C) 1989, 1990 Free Software Foundation, Inc.
+;; Copyright (C) 1989, 1990, 1994, 1998 Free Software Foundation, Inc.
 
 ;; Author: Neil Mager <neilm@juliet.ll.mit.edu>
-;; Version: 2.1
+;; Maintainer: FSF
 ;; Keywords: calendar
 
 ;; This file is part of GNU Emacs.
 ;; GNU General Public License for more details.
 
 ;; You should have received a copy of the GNU General Public License
-;; along with GNU Emacs; see the file COPYING.  If not, write to
-;; the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.
+;; along with GNU Emacs; see the file COPYING.  If not, write to the
+;; Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+;; Boston, MA 02111-1307, USA.
 
 ;;; Commentary:
 
 ;;
 ;; appt.el - visible and/or audible notification of
-;;           appointments from ~/diary file generated from
-;;           Edward M. Reingold's calendar.el.
+;;           appointments from ~/diary file.
 ;;
-;;
-;; Comments, corrections, and improvements should be sent to
-;; Neil M. Mager
-;; Net                     <neilm@juliet.ll.mit.edu>
-;; Voice                   (617) 981-4803
 ;;;
-;;; Thanks to  Edward M. Reingold for much help and many suggestions, 
+;;; Thanks to  Edward M. Reingold for much help and many suggestions,
 ;;; And to many others for bug fixes and suggestions.
 ;;;
 ;;;
 ;;; This functions in this file will alert the user of a 
 ;;; pending appointment based on their diary file.
 ;;;
-;;;
-;;; ******* It is necessary to invoke 'display-time' ********
-;;; *******  and 'diary' for this to work properly.  ********
-;;; 
-;;; A message will be displayed in the mode line of the emacs buffer
-;;; and (if the user desires) the terminal will beep and display a message
-;;; from the diary in the mini-buffer, or the user may select to 
+;;; A message will be displayed in the mode line of the Emacs buffer
+;;; and (if you request) the terminal will beep and display a message
+;;; from the diary in the mini-buffer, or you can choose to 
 ;;; have a message displayed in a new buffer.
 ;;;
-;;; The variable 'appt-message-warning-time' allows the
+;;; The variable `appt-message-warning-time' allows the
 ;;; user to specify how much notice they want before the appointment. The 
-;;; variable 'appt-issue-message' specifies whether the user wants
+;;; variable `appt-issue-message' specifies whether the user wants
 ;;; to to be notified of a pending appointment.
 ;;; 
-;;; In order to use, the following should be in your .emacs file in addition to
-;;; creating a diary file and invoking calendar:
-;;;
-;;;    Set some options
-;;; (setq view-diary-entries-initially t)
-;;; (setq appt-issue-message t)
-;;;
-;;;   The following three lines are required:
-;;; (display-time)
-;;; (autoload 'appt-make-list "appt.el" nil t)
-;;; (setq diary-display-hook 
-;;;     (list 'appt-make-list 'prepare-fancy-diary-buffer))
+;;; In order to use the appt package, you only need
+;;; to load it---provided you have appointments.
 ;;;
+;;; Before that, you can also set some options if you want
+;;;   (setq view-diary-entries-initially t)
+;;;   (setq appt-issue-message t)
 ;;; 
 ;;;  This is an example of what can be in your diary file:
 ;;; Monday
@@ -77,7 +61,7 @@
 ;;; 
 ;;; Based upon the above lines in your .emacs and diary files, 
 ;;; the calendar and diary will be displayed when you enter
-;;; emacs and your appointments list will automatically be created.
+;;; Emacs and your appointments list will automatically be created.
 ;;; You will then be reminded at 9:20am about your coffee break
 ;;; and at 11:50am to go to lunch. 
 ;;;
 ;;;  at 12:01am for those who do not logout every day or are programming
 ;;;  late.
 ;;;
-;;; Brief internal description - Skip this if your not interested!
-;;;
-;;; The function appt-check is run from the 'loadst' process which is started
-;;; by invoking (display-time). A temporary function below modifies
-;;; display-time-filter 
-;;; (from original time.el) to include a hook which will invoke appt-check.
-;;; This will not be necessary in the next version of gnuemacs.
-;;;
+;;; Brief internal description - Skip this if you are not interested!
 ;;;
 ;;; The function appt-make-list creates the appointments list which appt-check
 ;;; reads. This is all done automatically.
 ;;;
 ;;; For instance, these variables can be set to functions that display
 ;;; appointments in pop-up frames, which are lowered or iconified after
-;;; appt-display-interval seconds.
+;;; appt-display-interval minutes.
 ;;;
 
 ;;; Code:
 
+;; Make sure calendar is loaded when we compile this.
+(require 'calendar)
+
+(provide 'appt)
+
 ;;;###autoload
-(defvar appt-issue-message t
+(defcustom appt-issue-message t
   "*Non-nil means check for appointments in the diary buffer.
 To be detected, the diary entry must have the time
-as the first thing on a line.")
+as the first thing on a line."
+  :type 'boolean
+  :group 'appt)
 
 ;;;###autoload
-(defvar appt-message-warning-time 10
-  "*Time in minutes before an appointment that the warning begins.")
+(defcustom appt-message-warning-time 12
+  "*Time in minutes before an appointment that the warning begins."
+  :type 'integer
+  :group 'appt)
 
 ;;;###autoload
-(defvar appt-audible t
-  "*Non-nil means beep to indicate appointment.")
+(defcustom appt-audible t
+  "*Non-nil means beep to indicate appointment."
+  :type 'boolean
+  :group 'appt)
 
 ;;;###autoload
-(defvar appt-visible t
-  "*Non-nil means display appointment message in echo area.")
+(defcustom appt-visible t
+  "*Non-nil means display appointment message in echo area."
+  :type 'boolean
+  :group 'appt)
 
 ;;;###autoload
-(defvar appt-display-mode-line t
-  "*Non-nil means display minutes to appointment and time on the mode line.")
+(defcustom appt-display-mode-line t
+  "*Non-nil means display minutes to appointment and time on the mode line."
+  :type 'boolean
+  :group 'appt)
 
 ;;;###autoload
-(defvar appt-msg-window t
-  "*Non-nil means display appointment message in another window.")
+(defcustom appt-msg-window t
+  "*Non-nil means display appointment message in another window."
+  :type 'boolean
+  :group 'appt)
 
 ;;;###autoload
-(defvar appt-display-duration 5
-  "*The number of seconds an appointment message is displayed.")
+(defcustom appt-display-duration 10
+  "*The number of seconds an appointment message is displayed."
+  :type 'integer
+  :group 'appt)
 
 ;;;###autoload
-(defvar appt-display-diary t
+(defcustom appt-display-diary t
   "*Non-nil means to display the next days diary on the screen. 
-This will occur at midnight when the appointment list is updated.")
+This will occur at midnight when the appointment list is updated."
+  :type 'boolean
+  :group 'appt)
 
 (defvar appt-time-msg-list nil
   "The list of appointments for today.
@@ -159,11 +157,13 @@ Use `appt-add' and `appt-delete' to add and delete appointments from list.
 The original list is generated from the today's `diary-entries-list'.
 The number before each time/message is the time in minutes from midnight.")
 
-(defconst max-time 1439
+(defconst appt-max-time 1439
   "11:59pm in minutes - number of minutes in a day minus 1.")
 
-(defvar appt-display-interval 1
-  "*Number of minutes to wait between checking the appointment list.")
+(defcustom appt-display-interval 3
+  "*Number of minutes to wait between checking the appointment list."
+  :type 'integer
+  :group 'appt)
   
 (defvar appt-buffer-name " *appt-buf*"
   "Name of the appointments buffer.")
@@ -174,6 +174,18 @@ The number before each time/message is the time in minutes from midnight.")
 (defvar appt-delete-window-function 'appt-delete-window
   "Function called to remove appointment window and buffer.")
 
+(defvar appt-mode-string nil
+  "String being displayed in the mode line saying you have an appointment.
+The actual string includes the amount of time till the appointment.")
+
+(defvar appt-prev-comp-time nil
+  "Time of day (mins since midnight) at which we last checked appointments.")
+
+(defvar appt-now-displayed nil
+  "Non-nil when we have started notifying about a appointment that is near.")
+
+(defvar appt-display-count nil)
+
 (defun appt-check ()
   "Check for an appointment and update the mode line.
 Note: the time must be the first thing in the line in the diary
@@ -188,166 +200,179 @@ Example:
               Thursday
                 11:45am Lunch meeting.
 
-The following variables control the action of the notification:
+Appointments are checked every `appt-display-interval' minutes.
+The following variables control appointment notification:
 
-appt-issue-message
-       If T, the diary buffer is checked for appointments.
+`appt-issue-message'
+       If t, the diary buffer is checked for appointments.
 
-appt-message-warning-time
+`appt-message-warning-time'
        Variable used to determine if appointment message
        should be displayed.
 
-appt-audible
+`appt-audible'
        Variable used to determine if appointment is audible.
        Default is t.
 
-appt-visible
+`appt-visible'
        Variable used to determine if appointment message should be
-       displayed in the mini-buffer. Default is t.
+       displayed in the mini-buffer.  Default is t.
 
-appt-msg-window
+`appt-msg-window'
        Variable used to determine if appointment message
-       should temporarily appear in another window. Mutually exclusive
-       to appt-visible.
+       should temporarily appear in another window.  Mutually exclusive
+       to `appt-visible'.
 
-appt-display-duration
+`appt-display-duration'
        The number of seconds an appointment message
        is displayed in another window.
 
-appt-display-interval
-       The number of minutes to wait between checking the appointments
-       list.
-
-appt-disp-window-function 
-       Function called to display appointment window. You can customize
+`appt-disp-window-function'
+       Function called to display appointment window.  You can customize
        appt.el by setting this variable to a function different from the
        one provided with this package.
   
-appt-delete-window-function 
+`appt-delete-window-function'
        Function called to remove appointment window and buffer.  You can
        customize appt.el by setting this variable to a function different
-       from the one provided with this package.
-
-This function is run from the loadst process for display time.
-Therefore, you need to have `(display-time)' in your .emacs file."
-
-
-  (if (or (= appt-display-interval 1)
-         ;; This is true every appt-display-interval minutes.
-         (= 0 (mod (/ (nth 1 (current-time)) 60) appt-display-interval)))
-      (let ((min-to-app -1)
-           (new-time ""))
-       (save-excursion
-
-         ;; Get the current time and convert it to minutes
-         ;; from midnight. ie. 12:01am = 1, midnight = 0.
-
-         (let* ((cur-hour(string-to-int 
-                          (substring (current-time-string) 11 13)))
-                (cur-min (string-to-int 
-                          (substring (current-time-string) 14 16)))
-                (cur-comp-time (+ (* cur-hour 60) cur-min)))
-
-           ;; If the time is 12:01am, we should update our 
-           ;; appointments to todays list.
-
-           (if (= cur-comp-time 1)
-               (if (and view-diary-entries-initially appt-display-diary)
-                   (diary)
-                 (let ((diary-display-hook 'appt-make-list))
-                   (diary))))
-
-           ;; If there are entries in the list, and the
-           ;; user wants a message issued
-           ;; get the first time off of the list
-           ;; and calculate the number of minutes until
-           ;; the appointment.
-
-           (if (and appt-issue-message appt-time-msg-list)
-               (let ((appt-comp-time (car (car (car appt-time-msg-list)))))
-                 (setq min-to-app (- appt-comp-time cur-comp-time))
-
-                 (while (and appt-time-msg-list 
-                             (< appt-comp-time cur-comp-time))
-                   (setq appt-time-msg-list (cdr appt-time-msg-list)) 
-                   (if appt-time-msg-list
-                       (setq appt-comp-time 
-                             (car (car (car appt-time-msg-list))))))
-
-                 ;; If we have an appointment between midnight and
-                 ;; 'appt-message-warning-time' minutes after midnight,
-                 ;; we must begin to issue a message before midnight.
-                 ;; Midnight is considered 0 minutes and 11:59pm is
-                 ;; 1439 minutes. Therefore we must recalculate the minutes
-                 ;; to appointment variable. It is equal to the number of 
-                 ;; minutes before midnight plus the number of 
-                 ;; minutes after midnight our appointment is.
-
-                 (if (and (< appt-comp-time appt-message-warning-time)
-                          (> (+ cur-comp-time appt-message-warning-time)
-                             max-time))
-                     (setq min-to-app (+ (- (1+ max-time) cur-comp-time))
-                           appt-comp-time))
-
-                 ;; issue warning if the appointment time is 
-                 ;; within appt-message-warning time
-
-                 (if (and (<= min-to-app appt-message-warning-time)
+       from the one provided with this package."
+
+  (let* ((min-to-app -1)
+        (new-time "")
+        (prev-appt-mode-string appt-mode-string)
+        (prev-appt-display-count (or appt-display-count 0))
+        ;; Non-nil means do a full check for pending appointments
+        ;; and display in whatever ways the user has selected.
+        ;; When no appointment is being displayed,
+        ;; we always do a full check.
+        (full-check
+         (or (not appt-now-displayed)
+             ;; This is true every appt-display-interval minutes.
+             (= 0 (mod prev-appt-display-count appt-display-interval))))
+        ;; Non-nil means only update the interval displayed in the mode line.
+        (mode-line-only
+         (and (not full-check) appt-now-displayed)))
+
+    (when (or full-check mode-line-only)
+      (save-excursion
+
+       ;; Get the current time and convert it to minutes
+       ;; from midnight. ie. 12:01am = 1, midnight = 0.
+
+       (let* ((now (decode-time))
+              (cur-hour (nth 2 now))
+              (cur-min (nth 1 now))
+              (cur-comp-time (+ (* cur-hour 60) cur-min)))
+
+         ;; At the first check in any given day, update our 
+         ;; appointments to today's list.
+
+         (if (or (null appt-prev-comp-time)
+                 (< cur-comp-time appt-prev-comp-time))
+             (condition-case nil
+                 (progn
+                   (if (and view-diary-entries-initially appt-display-diary)
+                       (diary)
+                     (let ((diary-display-hook 'appt-make-list))
+                       (diary))))
+               (error nil)))
+         (setq appt-prev-comp-time cur-comp-time)
+
+         (setq appt-mode-string nil)
+         (setq appt-display-count nil)
+
+         ;; If there are entries in the list, and the
+         ;; user wants a message issued,
+         ;; get the first time off of the list
+         ;; and calculate the number of minutes until the appointment.
+
+         (if (and appt-issue-message appt-time-msg-list)
+             (let ((appt-comp-time (car (car (car appt-time-msg-list)))))
+               (setq min-to-app (- appt-comp-time cur-comp-time))
+
+               (while (and appt-time-msg-list 
+                           (< appt-comp-time cur-comp-time))
+                 (setq appt-time-msg-list (cdr appt-time-msg-list)) 
+                 (if appt-time-msg-list
+                     (setq appt-comp-time 
+                           (car (car (car appt-time-msg-list))))))
+
+               ;; If we have an appointment between midnight and
+               ;; 'appt-message-warning-time' minutes after midnight,
+               ;; we must begin to issue a message before midnight.
+               ;; Midnight is considered 0 minutes and 11:59pm is
+               ;; 1439 minutes. Therefore we must recalculate the minutes
+               ;; to appointment variable. It is equal to the number of 
+               ;; minutes before midnight plus the number of 
+               ;; minutes after midnight our appointment is.
+
+               (if (and (< appt-comp-time appt-message-warning-time)
+                        (> (+ cur-comp-time appt-message-warning-time)
+                           appt-max-time))
+                   (setq min-to-app (+ (- (1+ appt-max-time) cur-comp-time))
+                         appt-comp-time))
+
+               ;; issue warning if the appointment time is 
+               ;; within appt-message-warning time
+
+               (when (and (<= min-to-app appt-message-warning-time)
                           (>= min-to-app 0))
-                     (progn
-                       (if appt-msg-window
-                           (progn
-                             (string-match
-                              "[0-9]?[0-9]:[0-9][0-9]\\(am\\|pm\\)?" 
-                              display-time-string)
-
-                             (setq new-time (substring display-time-string 
-                                                       (match-beginning 0)
-                                                       (match-end 0)))
-                             (funcall
-                              appt-disp-window-function
-                              min-to-app new-time
-                              (car (cdr (car appt-time-msg-list))))
-                             
-                             (run-at-time
-                              (format "%d sec" appt-display-duration)
-                              nil
-                              appt-delete-window-function))
-                         ;;; else
-
-                         (if appt-visible
-                             (message "%s" 
-                                      (car (cdr (car appt-time-msg-list)))))
-
-                         (if appt-audible
-                             (beep 1)))
-
-                       (if appt-display-mode-line
-                           (progn
-                             (string-match
-                              "[0-9]?[0-9]:[0-9][0-9]\\(am\\|pm\\)?" 
-                              display-time-string)
-
-                             (setq new-time (substring display-time-string 
-                                                       (match-beginning 0)
-                                                       (match-end 0)))
-                             (setq display-time-string
-                                   (concat  "App't in "
-                                            min-to-app " min. " new-time " "))
-
-                             ;; force mode line updates - from time.el
-
-                             (save-excursion (set-buffer (other-buffer)))
-                             (set-buffer-modified-p (buffer-modified-p))
-                             (sit-for 0)))
-
-                       (if (= min-to-app 0)
-                           (setq appt-time-msg-list
-                                 (cdr appt-time-msg-list))))))))))))
-
-
-;; Display appointment message in a separate buffer.
+                 (setq appt-now-displayed t)
+                 (setq appt-display-count
+                       (1+ prev-appt-display-count))
+                 (unless mode-line-only
+                   (if appt-msg-window
+                       (progn
+                         (setq new-time (format-time-string "%a %b %e "
+                                                            (current-time)))
+                         (funcall
+                          appt-disp-window-function
+                          (number-to-string min-to-app) new-time
+                          (car (cdr (car appt-time-msg-list))))
+
+                         (run-at-time
+                          (format "%d sec" appt-display-duration)
+                          nil
+                          appt-delete-window-function))
+                             ;;; else
+
+                     (if appt-visible
+                         (message "%s" 
+                                  (car (cdr (car appt-time-msg-list)))))
+
+                     (if appt-audible
+                         (beep 1))))
+
+                 (when appt-display-mode-line
+                   (setq appt-mode-string
+                         (concat  " App't in "
+                                  (number-to-string min-to-app)
+                                  " min. ")))
+
+                 ;; When an appointment is reached,
+                 ;; delete it from the list.
+                 ;; Reset the count to 0 in case we display another
+                 ;; appointment on the next cycle.
+                 (if (= min-to-app 0)
+                     (setq appt-time-msg-list
+                           (cdr appt-time-msg-list)
+                           appt-display-count nil)))))
+
+         ;; If we have changed the mode line string,
+         ;; redisplay all mode lines.
+         (and appt-display-mode-line
+              (not (equal appt-mode-string
+                          prev-appt-mode-string))
+              (progn
+                (force-mode-line-update t)
+                ;; If the string now has a notification,
+                ;; redisplay right now.
+                (if appt-mode-string
+                    (sit-for 0)))))))))
+
+
 (defun appt-disp-window (min-to-app new-time appt-msg)
+  "Display appointment message APPT-MSG in a separate buffer."
   (require 'electric)
 
   ;; Make sure we're not in the minibuffer
@@ -356,23 +381,30 @@ Therefore, you need to have `(display-time)' in your .emacs file."
   (if (equal (selected-window) (minibuffer-window))
       (if (other-window 1) 
          (select-window (other-window 1))
-       (if window-system
+       (if (display-multi-frame-p)
            (select-frame (other-frame 1)))))
       
   (let* ((this-buffer (current-buffer))
         (this-window (selected-window))
         (appt-disp-buf (set-buffer (get-buffer-create appt-buffer-name))))
 
-    (appt-select-lowest-window)
-    (split-window)
-      
-    (pop-to-buffer appt-disp-buf)
+    (if (cdr (assq 'unsplittable (frame-parameters)))
+       ;; In an unsplittable frame, use something somewhere else.
+       (display-buffer appt-disp-buf)
+      (unless (or (special-display-p (buffer-name appt-disp-buf))
+                 (same-window-p (buffer-name appt-disp-buf)))
+       ;; By default, split the bottom window and use the lower part.
+       (appt-select-lowest-window)
+       (split-window))
+      (pop-to-buffer appt-disp-buf))
     (setq mode-line-format 
          (concat "-------------------- Appointment in "
                  min-to-app " minutes. " new-time " %-"))
+    (erase-buffer)
     (insert-string appt-msg)
-    (shrink-window-if-larger-than-buffer (get-buffer-window appt-disp-buf))
+    (shrink-window-if-larger-than-buffer (get-buffer-window appt-disp-buf t))
     (set-buffer-modified-p nil)
+    (raise-frame (selected-frame))
     (select-window this-window)
     (if appt-audible
        (beep 1))))
@@ -380,35 +412,28 @@ Therefore, you need to have `(display-time)' in your .emacs file."
 (defun appt-delete-window ()
   "Function called to undisplay appointment messages.
 Usually just deletes the appointment buffer."
-  (delete-window (get-buffer-window appt-buffer-name))
+  (let ((window (get-buffer-window appt-buffer-name t)))
+    (and window
+        (or (eq window (frame-root-window (window-frame window)))
+            (delete-window window))))
   (kill-buffer appt-buffer-name)
   (if appt-audible
       (beep 1)))
 
-;; Select the lowest window on the frame.
 (defun appt-select-lowest-window ()
-  (setq lowest-window (selected-window))
-  (let* ((bottom-edge (car (cdr (cdr (cdr (window-edges))))))
-         (last-window (previous-window))
-         (window-search t))
-    (while window-search
-      (let* ((this-window (next-window))
-             (next-bottom-edge (car (cdr (cdr (cdr 
-                                               (window-edges this-window)))))))
-        (if (< bottom-edge next-bottom-edge)
-            (progn
-              (setq bottom-edge next-bottom-edge)
-              (setq lowest-window this-window)))
-
-        (select-window this-window)
-        (if (eq last-window this-window)
-            (progn
-              (select-window lowest-window)
-              (setq window-search nil)))))))
-
+"Select the lowest window on the frame."
+  (let ((lowest-window (selected-window))
+       (bottom-edge (nth 3 (window-edges))))
+    (walk-windows (lambda (w)
+                   (let ((next-bottom-edge (nth 3 (window-edges w))))
+                     (when (< bottom-edge next-bottom-edge)
+                       (setq bottom-edge next-bottom-edge
+                             lowest-window w)))))
+    (select-window lowest-window)))
 
+;;;###autoload
 (defun appt-add (new-appt-time new-appt-msg)
-  "Add an appointment for the day at TIME and issue MESSAGE.
+  "Add an appointment for the day at NEW-APPT-TIME and issue message NEW-APPT-MSG.
 The time should be in either 24 hour format or am/pm format."
 
   (interactive "sTime (hh:mm[am/pm]): \nsMessage: ")
@@ -419,10 +444,10 @@ The time should be in either 24 hour format or am/pm format."
   (let* ((appt-time-string (concat new-appt-time " " new-appt-msg))
          (appt-time (list (appt-convert-time new-appt-time)))
          (time-msg (cons appt-time (list appt-time-string))))
-    (setq appt-time-msg-list (append appt-time-msg-list
-                                     (list time-msg)))
+    (setq appt-time-msg-list (nconc appt-time-msg-list (list time-msg)))
     (setq appt-time-msg-list (appt-sort-list appt-time-msg-list)))) 
 
+;;;###autoload
 (defun appt-delete ()
   "Delete an appointment from the list of appointments."
   (interactive)
@@ -435,91 +460,108 @@ The time should be in either 24 hour format or am/pm format."
              (test-input (y-or-n-p prompt-string)))
         (setq tmp-msg-list (cdr tmp-msg-list))
         (if test-input
-            (setq appt-time-msg-list (delq element appt-time-msg-list)))
-        (setq tmp-appt-msg-list nil)))
+            (setq appt-time-msg-list (delq element appt-time-msg-list)))))
+    (appt-check)
     (message "")))
                  
 
-;; Create the appointments list from todays diary buffer.
-;; The time must be at the beginning of a line for it to be
-;; put in the appointments list.
-;;                02/23/89
-;;                  12:00pm lunch
-;;                 Wednesday
-;;                   10:00am group meeting"
-
+(eval-when-compile (defvar number)
+                  (defvar original-date)
+                  (defvar diary-entries-list))
 ;;;###autoload
 (defun appt-make-list ()
-  (setq appt-time-msg-list nil)
-
-  (save-excursion
-    (if diary-entries-list
-
-        ;; Cycle through the entry-list (diary-entries-list)
-        ;; looking for entries beginning with a time. If 
-        ;; the entry begins with a time, add it to the
-        ;; appt-time-msg-list. Then sort the list.
-        
-        (let ((entry-list diary-entries-list)
-              (new-time-string ""))
-          (while (and entry-list 
-                      (calendar-date-equal 
-                       (calendar-current-date) (car (car entry-list))))
-            (let ((time-string (substring (prin1-to-string 
-                                           (cdr (car entry-list))) 2 -2)))
-              
-              (while (string-match
-                      "[0-9]?[0-9]:[0-9][0-9]\\(am\\|pm\\)?.*" 
-                      time-string)
-                (let* ((appt-time-string (substring time-string
-                                                    (match-beginning 0)
-                                                    (match-end 0))))
-
-                  (if (< (match-end 0) (length time-string))
-                      (setq new-time-string (substring time-string 
-                                                       (+ (match-end 0) 1)
-                                                       nil))
-                    (setq new-time-string ""))
-                  
-                  (string-match "[0-9]?[0-9]:[0-9][0-9]\\(am\\|pm\\)?"
-                                time-string)
-                    
-                  (let* ((appt-time (list (appt-convert-time 
-                                           (substring time-string
-                                                      (match-beginning 0)
-                                                      (match-end 0)))))
-                         (time-msg (cons appt-time
-                                         (list appt-time-string))))
-                    (setq time-string new-time-string)
-                    (setq appt-time-msg-list (append appt-time-msg-list
-                                                     (list time-msg)))))))
-            (setq entry-list (cdr entry-list)))))
-    (setq appt-time-msg-list (appt-sort-list appt-time-msg-list))
-
-    ;; Get the current time and convert it to minutes
-    ;; from midnight. ie. 12:01am = 1, midnight = 0,
-    ;; so that the elements in the list
-    ;; that are earlier than the present time can
-    ;; be removed.
-  
-    (let* ((cur-hour(string-to-int 
-                    (substring (current-time-string) 11 13)))
-          (cur-min (string-to-int 
-                    (substring (current-time-string) 14 16)))
-          (cur-comp-time (+ (* cur-hour 60) cur-min))
-          (appt-comp-time (car (car (car appt-time-msg-list)))))
-
-      (while (and appt-time-msg-list (< appt-comp-time cur-comp-time))
-       (setq appt-time-msg-list (cdr appt-time-msg-list)) 
-       (if appt-time-msg-list
-           (setq appt-comp-time (car (car (car appt-time-msg-list)))))))))
+  "Create the appointments list from todays diary buffer.
+The time must be at the beginning of a line for it to be
+put in the appointments list.
+  02/23/89
+    12:00pm lunch
+   Wednesday
+     10:00am group meeting
+We assume that the variables DATE and NUMBER
+hold the arguments that `list-diary-entries' received.
+They specify the range of dates that the diary is being processed for."
+
+  ;; We have something to do if the range of dates that the diary is
+  ;; considering includes the current date.
+  (if (and (not (calendar-date-compare
+                (list (calendar-current-date))
+                (list original-date)))
+          (calendar-date-compare
+           (list (calendar-current-date))
+            (list (calendar-gregorian-from-absolute
+                  (+ (calendar-absolute-from-gregorian original-date)
+                     number)))))
+      (save-excursion
+       ;; Clear the appointments list, then fill it in from the diary.
+       (setq appt-time-msg-list nil)
+       (if diary-entries-list
+
+           ;; Cycle through the entry-list (diary-entries-list)
+           ;; looking for entries beginning with a time. If 
+           ;; the entry begins with a time, add it to the
+           ;; appt-time-msg-list. Then sort the list.
+
+           (let ((entry-list diary-entries-list)
+                 (new-time-string ""))
+             ;; Skip diary entries for dates before today.
+             (while (and entry-list
+                         (calendar-date-compare
+                          (car entry-list) (list (calendar-current-date))))
+               (setq entry-list (cdr entry-list)))
+             ;; Parse the entries for today.
+             (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)))
+
+                 (while (string-match
+                         "[0-9]?[0-9]:[0-9][0-9]\\(am\\|pm\\)?\\(.*\n\\)*.*"
+                         time-string)
+                   (let* ((appt-time-string (match-string 0 time-string)))
+
+                     (if (< (match-end 0) (length time-string))
+                         (setq new-time-string (substring time-string 
+                                                          (+ (match-end 0) 1)
+                                                          nil))
+                       (setq new-time-string ""))
+
+                     (string-match "[0-9]?[0-9]:[0-9][0-9]\\(am\\|pm\\)?"
+                                   time-string)
+
+                     (let* ((appt-time (list (appt-convert-time 
+                                              (match-string 0 time-string))))
+                            (time-msg (cons appt-time
+                                            (list appt-time-string))))
+                       (setq time-string new-time-string)
+                       (setq appt-time-msg-list (nconc appt-time-msg-list
+                                                       (list time-msg)))))))
+               (setq entry-list (cdr entry-list)))))
+       (setq appt-time-msg-list (appt-sort-list appt-time-msg-list))
+
+       ;; Get the current time and convert it to minutes
+       ;; from midnight. ie. 12:01am = 1, midnight = 0,
+       ;; so that the elements in the list
+       ;; that are earlier than the present time can
+       ;; be removed.
+
+       (let* ((now (decode-time))
+              (cur-hour (nth 2 now))
+              (cur-min (nth 1 now))
+              (cur-comp-time (+ (* cur-hour 60) cur-min))
+              (appt-comp-time (car (car (car appt-time-msg-list)))))
+
+         (while (and appt-time-msg-list (< appt-comp-time cur-comp-time))
+           (setq appt-time-msg-list (cdr appt-time-msg-list)) 
+           (if appt-time-msg-list
+               (setq appt-comp-time (car (car (car appt-time-msg-list))))))))))
   
 
-;;Simple sort to put the appointments list in order.
-;;Scan the list for the smallest element left in the list.
-;;Append the smallest element left into the new list, and remove
-;;it from the original list.
 (defun appt-sort-list (appt-list)
+  "Simple sort to put the appointments list APPT-LIST in order.
+Scan the list for the smallest element left in the list.
+Append the smallest element left into the new list, and remove
+it from the original list."
   (let ((order-list nil))
     (while appt-list
       (let* ((element (car appt-list))
@@ -531,7 +573,7 @@ The time should be in either 24 hour format or am/pm format."
             (setq element (car tmp-list))
             (setq element-time (car (car element))))
           (setq tmp-list (cdr tmp-list)))
-        (setq order-list (append order-list (list element)))
+        (setq order-list (nconc order-list (list element)))
         (setq appt-list (delq element appt-list))))
     order-list))
 
@@ -543,23 +585,20 @@ The time should be in either 24 hour format or am/pm format."
         (hr 0)
         (min 0))
 
-    (string-match ":[0-9][0-9]" time2conv)
+    (string-match ":\\([0-9][0-9]\\)" time2conv)
     (setq min (string-to-int 
-               (substring time2conv 
-                          (+ (match-beginning 0) 1) (match-end 0))))
+               (match-string 1 time2conv)))
   
     (string-match "[0-9]?[0-9]:" time2conv)
     (setq hr (string-to-int 
-              (substring time2conv 
-                         (match-beginning 0)
-                         (match-end 0))))
+              (match-string 0 time2conv)))
   
     ;; convert the time appointment time into 24 hour time
   
-    (if (and (string-match  "[p][m]" time2conv) (< hr 12))
-        (progn
-          (string-match "[0-9]?[0-9]:" time2conv)
-          (setq hr (+ 12 hr))))
+    (cond ((and (string-match "pm" time2conv) (< hr 12))
+          (setq hr (+ 12 hr)))
+         ((and (string-match "am" time2conv) (= hr 12))
+           (setq hr 0)))
   
     ;; convert the actual time
     ;; into minutes for comparison
@@ -568,7 +607,15 @@ The time should be in either 24 hour format or am/pm format."
     (setq conv-time (+ (* hr 60) min))
     conv-time))
 
-(setq display-time-hook 'appt-check)
+(defvar appt-timer nil
+  "Timer used for diary appointment notifications (`appt-check').")
 
-;;; appt.el ends here
+(unless appt-timer
+  (setq appt-timer (run-at-time t 60 'appt-check)))
+
+(or global-mode-string (setq global-mode-string '("")))
+(or (memq 'appt-mode-string global-mode-string)
+    (setq global-mode-string
+         (append global-mode-string '(appt-mode-string))))
 
+;;; appt.el ends here