Fix bug#6999.
[bpt/emacs.git] / lisp / calendar / appt.el
index 94df068..0818492 100644 (file)
@@ -1,7 +1,7 @@
 ;;; appt.el --- appointment notification functions
 
 ;; Copyright (C) 1989, 1990, 1994, 1998, 2001, 2002, 2003, 2004, 2005,
-;;   2006, 2007  Free Software Foundation, Inc.
+;;   2006, 2007, 2008, 2009, 2010  Free Software Foundation, Inc.
 
 ;; Author: Neil Mager <neilm@juliet.ll.mit.edu>
 ;; Maintainer: Glenn Morris <rgm@gnu.org>
@@ -9,10 +9,10 @@
 
 ;; This file is part of GNU Emacs.
 
-;; GNU Emacs is free software; you can redistribute it and/or modify
+;; GNU Emacs is free software: you can redistribute it and/or modify
 ;; it under the terms of the GNU General Public License as published by
-;; the Free Software Foundation; either version 3, or (at your option)
-;; any later version.
+;; the Free Software Foundation, either version 3 of the License, or
+;; (at your option) any later version.
 
 ;; GNU Emacs is distributed in the hope that it will be useful,
 ;; but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ -20,9 +20,7 @@
 ;; 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, Inc., 51 Franklin Street, Fifth Floor,
-;; Boston, MA 02110-1301, USA.
+;; along with GNU Emacs.  If not, see <http://www.gnu.org/licenses/>.
 
 ;;; Commentary:
 
 ;; appt.el - visible and/or audible notification of
 ;;           appointments from diary file.
 ;;
-;;;
-;;; 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 his/her diary file.  This package
-;;; is documented in the Emacs manual.
-;;;
-;;; To activate this package, simply use (appt-activate 1).
-;;; A `diary-file' with appointments of the format described in the
-;;; documentation of the function `appt-check' is required.
-;;; Relevant customizable variables are also listed in the
-;;; documentation of that function.
-;;;
-;;; Today's appointment list is initialized from the diary when this
-;;; package is activated. Additionally, the appointments list is
-;;; recreated automatically at 12:01am for those who do not logout
-;;; every day or are programming late. It is also updated when the
-;;; `diary-file' is saved. Calling `appt-check' with an argument forces
-;;; a re-initialization at any time.
-;;;
-;;; In order to add or delete items from today's list, without
-;;; changing the diary file, use `appt-add' and `appt-delete'.
-;;;
-
-;;; Brief internal description - Skip this if you are not interested!
-;;;
-;;; The function `appt-make-list' creates the appointments list which
-;;; `appt-check' reads.
-;;;
-;;; You can change the way the appointment window is created/deleted by
-;;; setting the variables
-;;;
-;;;         appt-disp-window-function
-;;; and
-;;;         appt-delete-window-function
-;;;
-;;; For instance, these variables could be set to functions that display
-;;; appointments in pop-up frames, which are lowered or iconified after
-;;; `appt-display-interval' minutes.
-;;;
+;;
+;; 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 his/her diary file.  This package
+;; is documented in the Emacs manual.
+;;
+;; To activate this package, simply use (appt-activate 1).
+;; A `diary-file' with appointments of the format described in the
+;; documentation of the function `appt-check' is required.
+;; Relevant customizable variables are also listed in the
+;; documentation of that function.
+;;
+;; Today's appointment list is initialized from the diary when this
+;; package is activated.  Additionally, the appointments list is
+;; recreated automatically at 12:01am for those who do not logout
+;; every day or are programming late.  It is also updated when the
+;; `diary-file' is saved.  Calling `appt-check' with an argument (or
+;; re-enabling the package) forces a re-initialization at any time.
+;;
+;; In order to add or delete items from today's list, without
+;; changing the diary file, use `appt-add' and `appt-delete'.
+;;
+
+;; Brief internal description - Skip this if you are not interested!
+;;
+;; The function `appt-make-list' creates the appointments list which
+;; `appt-check' reads.
+;;
+;; You can change the way the appointment window is created/deleted by
+;; setting the variables
+;;
+;;           appt-disp-window-function
+;; and
+;;           appt-delete-window-function
+;;
+;; For instance, these variables could be set to functions that display
+;; appointments in pop-up frames, which are lowered or iconified after
+;; `appt-display-interval' minutes.
+;;
 
 ;;; Code:
 
-;; Make sure calendar is loaded when we compile this.
-(require 'calendar)
+(require 'diary-lib)
 
-(defvar diary-selective-display)
 
-;;;###autoload
+(defgroup appt nil
+  "Appointment notification."
+  :prefix "appt-"
+  :group 'calendar)
+
 (defcustom appt-issue-message t
-  "*Non-nil means check for appointments in the diary buffer.
+  "Non-nil means check for appointments in the diary buffer.
 To be detected, the diary entry must have the format described in the
 documentation of the function `appt-check'."
   :type 'boolean
@@ -92,30 +92,26 @@ documentation of the function `appt-check'."
                         "use the function `appt-activate', and the \
 variable `appt-display-format' instead." "22.1")
 
-;;;###autoload
 (defcustom appt-message-warning-time 12
-  "*Time in minutes before an appointment that the warning begins."
+  "Time in minutes before an appointment that the warning begins."
   :type 'integer
   :group 'appt)
 
-;;;###autoload
 (defcustom appt-audible t
-  "*Non-nil means beep to indicate appointment."
+  "Non-nil means beep to indicate appointment."
   :type 'boolean
   :group 'appt)
 
-;;;###autoload
 (defcustom appt-visible t
-  "*Non-nil means display appointment message in echo area.
+  "Non-nil means display appointment message in echo area.
 This variable is only relevant if `appt-msg-window' is nil."
   :type 'boolean
   :group 'appt)
 
 (make-obsolete-variable 'appt-visible 'appt-display-format "22.1")
 
-;;;###autoload
 (defcustom appt-msg-window t
-  "*Non-nil means display appointment message in another window.
+  "Non-nil means display appointment message in another window.
 If non-nil, this variable overrides `appt-visible'."
   :type 'boolean
   :group 'appt)
@@ -142,35 +138,34 @@ of the (obsolete) variables `appt-msg-window' and `appt-visible'."
   :group 'appt
   :version "22.1")
 
-;;;###autoload
 (defcustom appt-display-mode-line t
-  "*Non-nil means display minutes to appointment and time on the mode line.
+  "Non-nil means display minutes to appointment and time on the mode line.
 This is in addition to any other display of appointment messages."
   :type 'boolean
   :group 'appt)
 
-;;;###autoload
 (defcustom appt-display-duration 10
-  "*The number of seconds an appointment message is displayed.
+  "The number of seconds an appointment message is displayed.
 Only relevant if reminders are to be displayed in their own window."
   :type 'integer
   :group 'appt)
 
-;;;###autoload
 (defcustom appt-display-diary t
-  "*Non-nil displays the diary when the appointment list is first initialized.
+  "Non-nil displays the diary when the appointment list is first initialized.
 This will occur at midnight when the appointment list is updated."
   :type 'boolean
   :group 'appt)
 
 (defcustom appt-display-interval 3
-  "*Number of minutes to wait between checking the appointment list."
+  "Number of minutes to wait between checking the appointment list."
   :type 'integer
   :group 'appt)
 
 (defcustom appt-disp-window-function 'appt-disp-window
   "Function called to display appointment window.
-Only relevant if reminders are being displayed in a window."
+Only relevant if reminders are being displayed in a window.
+It should take three string arguments: the number of minutes till
+the appointment, the current time, and the text of the appointment."
   :type '(choice (const appt-disp-window)
                  function)
   :group 'appt)
@@ -185,19 +180,28 @@ Only relevant if reminders are being displayed in a window."
 
 ;;; Internal variables below this point.
 
-(defconst appt-buffer-name " *appt-buf*"
+(defconst appt-buffer-name "*appt-buf*"
   "Name of the appointments buffer.")
 
+;; TODO Turn this into an alist?  It would be easier to add more
+;; optional elements.
+;; TODO There should be a way to set WARNTIME (and other properties)
+;; from the diary-file.  Implementing that would be a good reason
+;; to change this to an alist.
 (defvar appt-time-msg-list nil
   "The list of appointments for today.
 Use `appt-add' and `appt-delete' to add and delete appointments.
 The original list is generated from today's `diary-entries-list', and
 can be regenerated using the function `appt-check'.
-Each element of the generated list has the form (MINUTES STRING [FLAG]); where
-MINUTES is the time in minutes of the appointment after midnight, and
-STRING is the description of the appointment.
-FLAG, if non-nil, says that the element was made with `appt-add'
-so calling `appt-make-list' again should preserve it.")
+Each element of the generated list has the form
+\(MINUTES STRING [FLAG] [WARNTIME])
+where MINUTES is the time in minutes of the appointment after midnight,
+and STRING is the description of the appointment.
+FLAG and WARNTIME can only be present if the element was made
+with `appt-add'.  A non-nil FLAG indicates that the element was made
+with `appt-add', so calling `appt-make-list' again should preserve it.
+If WARNTIME is non-nil, it is an integer to use in place
+of `appt-message-warning-time'.")
 
 (defconst appt-max-time (1- (* 24 60))
   "11:59pm in minutes - number of minutes in a day minus 1.")
@@ -206,6 +210,7 @@ so calling `appt-make-list' again should preserve it.")
   "String being displayed in the mode line saying you have an appointment.
 The actual string includes the amount of time till the appointment.
 Only used if `appt-display-mode-line' is non-nil.")
+(put 'appt-mode-string 'risky-local-variable t) ; for 'face property
 
 (defvar appt-prev-comp-time nil
   "Time of day (mins since midnight) at which we last checked appointments.
@@ -229,27 +234,28 @@ If this is non-nil, appointment checking is active.")
 The string STRING describes the appointment, due in integer MINS minutes.
 The format of the visible reminder is controlled by `appt-display-format'.
 The variable `appt-audible' controls the audible reminder."
-  ;; let binding for backwards compatability. Remove when obsolete
+  ;; Let-binding for backwards compatibility.  Remove when obsolete
   ;; vars appt-msg-window and appt-visible are dropped.
   (let ((appt-display-format
          (if (eq appt-display-format 'ignore)
-               (cond (appt-msg-window 'window)
-                     (appt-visible 'echo))
+             (cond (appt-msg-window 'window)
+                   (appt-visible 'echo))
            appt-display-format)))
+    (if appt-audible (beep 1))
     (cond ((eq appt-display-format 'window)
            (funcall appt-disp-window-function
                     (number-to-string mins)
-                    ;; TODO - use calendar-month-abbrev-array rather
-                    ;; than %b?
+                    ;; TODO - use calendar-month-abbrev-array rather than %b?
                     (format-time-string "%a %b %e " (current-time))
                     string)
            (run-at-time (format "%d sec" appt-display-duration)
                         nil
                         appt-delete-window-function))
           ((eq appt-display-format 'echo)
-           (message "%s" string)))
-    (if appt-audible (beep 1))))
+           (message "%s" string)))))
+
 
+(defvar diary-selective-display)
 
 (defun appt-check (&optional force)
   "Check for an appointment and update any reminder display.
@@ -274,12 +280,12 @@ The following variables control appointment notification:
         Controls the format in which reminders are displayed.
 
 `appt-audible'
-       Variable used to determine if reminder is audible.
-       Default is t.
+        Variable used to determine if reminder is audible.
+        Default is t.
 
 `appt-message-warning-time'
-       Variable used to determine when appointment message
-       should first be displayed.
+        Variable used to determine when appointment message
+        should first be displayed.
 
 `appt-display-mode-line'
         If non-nil, a generic message giving the time remaining
@@ -296,177 +302,157 @@ The following variables are only relevant if reminders are being
 displayed in a window:
 
 `appt-display-duration'
-       The number of seconds an appointment message is displayed.
+        The number of seconds an appointment message is displayed.
 
 `appt-disp-window-function'
-       Function called to display appointment window.
+        Function called to display appointment window.
 
 `appt-delete-window-function'
-       Function called to remove appointment window and buffer."
-
+        Function called to remove appointment window and buffer."
+  (interactive "P")                     ; so people can force updates
   (let* ((min-to-app -1)
-        (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.
-             (zerop (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)))
-
+         (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.
+              (zerop (mod prev-appt-display-count appt-display-interval))))
+         ;; Non-nil means only update the interval displayed in the mode line.
+         (mode-line-only (unless full-check appt-now-displayed))
+         now cur-comp-time appt-comp-time appt-warn-time)
     (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 force                 ; eg initialize, diary save
-                  (null appt-prev-comp-time)             ; first check
-                 (< cur-comp-time appt-prev-comp-time)) ; new day
-             (condition-case nil
-                  (if appt-display-diary
-                      (let ((diary-hook
-                             (if (assoc 'appt-make-list diary-hook)
-                                 diary-hook
-                               (cons 'appt-make-list diary-hook))))
-                        (diary))
-                    (let* ((diary-display-hook 'appt-make-list)
-                           (d-buff (find-buffer-visiting
-                                    (substitute-in-file-name diary-file)))
-                           (selective
-                            (if d-buff        ; Diary buffer exists.
-                                (with-current-buffer d-buff
-                                  diary-selective-display))))
-                      (diary)
-                      ;; If the diary buffer existed before this command,
-                      ;; restore its display state. Otherwise, kill it.
-                      (if d-buff
-                          ;; Displays the diary buffer.
-                          (or selective (diary-show-all-entries))
-                        (and
-                         (setq d-buff (find-buffer-visiting
-                                       (substitute-in-file-name diary-file)))
-                         (kill-buffer d-buff)))))
-               (error nil)))
-
-         (setq appt-prev-comp-time cur-comp-time
-                appt-mode-string nil
-                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))
-                 (setq appt-now-displayed t
-                        appt-display-count (1+ prev-appt-display-count))
-                 (unless mode-line-only
-                    (appt-display-message (cadr (car appt-time-msg-list))
-                                          min-to-app))
-                 (when appt-display-mode-line
-                   (setq appt-mode-string
-                          (format " App't in %s min." min-to-app)))
-
-                 ;; 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 (zerop min-to-app)
-                     (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)))))))))
-
+        ;; Convert current time to minutes after midnight (12.01am = 1).
+        (setq now (decode-time)
+              cur-comp-time (+ (* 60 (nth 2 now)) (nth 1 now)))
+        ;; At first check in any day, update appointments to today's list.
+        (if (or force                      ; eg initialize, diary save
+                (null appt-prev-comp-time) ; first check
+                (< cur-comp-time appt-prev-comp-time)) ; new day
+            (condition-case nil
+                (if appt-display-diary
+                    (let ((diary-hook
+                           (if (assoc 'appt-make-list diary-hook)
+                               diary-hook
+                             (cons 'appt-make-list diary-hook))))
+                      (diary))
+                  (let* ((diary-display-function 'appt-make-list)
+                         (d-buff (find-buffer-visiting diary-file))
+                         (selective
+                          (if d-buff    ; diary buffer exists
+                              (with-current-buffer d-buff
+                                diary-selective-display))))
+                    ;; FIXME why not using diary-list-entries with
+                    ;; non-nil LIST-ONLY?
+                    (diary)
+                    ;; If the diary buffer existed before this command,
+                    ;; restore its display state.  Otherwise, kill it.
+                    (if d-buff
+                        ;; Displays the diary buffer.
+                        (or selective (diary-show-all-entries))
+                      (and (setq d-buff (find-buffer-visiting diary-file))
+                           (kill-buffer d-buff)))))
+              (error nil)))
+        (setq appt-prev-comp-time cur-comp-time
+              appt-mode-string nil
+              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.
+        (when (and appt-issue-message appt-time-msg-list)
+          (setq appt-comp-time (caar (car appt-time-msg-list))
+                appt-warn-time (or (nth 3 (car appt-time-msg-list))
+                                   appt-message-warning-time)
+                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 (caar (car appt-time-msg-list)))))
+          ;; If we have an appointment between midnight and
+          ;; `appt-warn-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-warn-time)
+                   (> (+ cur-comp-time appt-warn-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-warn-time)
+                     (>= min-to-app 0))
+            (setq appt-now-displayed t
+                  appt-display-count (1+ prev-appt-display-count))
+            (unless mode-line-only
+              (appt-display-message (cadr (car appt-time-msg-list))
+                                    min-to-app))
+            (when appt-display-mode-line
+              (setq appt-mode-string
+                    (concat " " (propertize
+                                 (format "App't in %s min." min-to-app)
+                                 'face 'mode-line-emphasis))))
+            ;; 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 (zerop min-to-app)
+                (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 (string-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.
-The appointment is due in MIN-TO-APP (a string) minutes.
-NEW-TIME is a string giving the date."
-  (require 'electric)
-
-  ;; Make sure we're not in the minibuffer
-  ;; before splitting the window.
-
-  (if (equal (selected-window) (minibuffer-window))
-      (if (other-window 1)
-         (select-window (other-window 1))
-       (if (display-multi-frame-p)
-           (select-frame (other-frame 1)))))
-
+  "Display appointment due in MIN-TO-APP (a string) minutes.
+NEW-TIME is a string giving the date.  Displays the appointment
+message APPT-MSG in a separate buffer."
   (let ((this-window (selected-window))
-        (appt-disp-buf (set-buffer (get-buffer-create appt-buffer-name))))
-
+        (appt-disp-buf (get-buffer-create appt-buffer-name)))
+    ;; Make sure we're not in the minibuffer before splitting the window.
+    ;; FIXME this seems needlessly complicated?
+    (when (minibufferp)
+      (other-window 1)
+      (and (minibufferp) (display-multi-frame-p) (other-frame 1)))
     (if (cdr (assq 'unsplittable (frame-parameters)))
-       ;; In an unsplittable frame, use something somewhere else.
-       (display-buffer appt-disp-buf)
+        ;; In an unsplittable frame, use something somewhere else.
+       (progn
+         (set-buffer appt-disp-buf)
+         (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)
+                  (same-window-p (buffer-name appt-disp-buf)))
+        ;; By default, split the bottom window and use the lower part.
+        (appt-select-lowest-window)
         ;; Split the window, unless it's too small to do so.
         (when (>= (window-height) (* 2 window-min-height))
           (select-window (split-window))))
       (switch-to-buffer appt-disp-buf))
+    ;; FIXME Link to diary entry?
     (calendar-set-mode-line
-     (format " Appointment in %s minutes. %s " min-to-app new-time))
+     (format " Appointment %s. %s "
+             (if (string-equal "0" min-to-app) "now"
+               (format "in %s minute%s" min-to-app
+                       (if (string-equal "1" min-to-app) "" "s")))
+             new-time))
+    (setq buffer-read-only nil
+          buffer-undo-list t)
     (erase-buffer)
     (insert appt-msg)
     (shrink-window-if-larger-than-buffer (get-buffer-window appt-disp-buf t))
     (set-buffer-modified-p nil)
+    (setq buffer-read-only t)
     (raise-frame (selected-frame))
     (select-window this-window)))
 
@@ -475,35 +461,50 @@ NEW-TIME is a string giving the date."
 Usually just deletes the appointment buffer."
   (let ((window (get-buffer-window appt-buffer-name t)))
     (and window
-        (or (eq window (frame-root-window (window-frame window)))
-            (delete-window window))))
+         (or (eq window (frame-root-window (window-frame window)))
+             (delete-window window))))
   (kill-buffer appt-buffer-name)
   (if appt-audible
       (beep 1)))
 
 (defun appt-select-lowest-window ()
-"Select the lowest window on the frame."
+  "Select the lowest window on the frame."
   (let ((lowest-window (selected-window))
-       (bottom-edge (nth 3 (window-edges))))
+        (bottom-edge (nth 3 (window-edges)))
+        next-bottom-edge)
     (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)))))
+                    (when (< bottom-edge (setq next-bottom-edge
+                                               (nth 3 (window-edges w))))
+                      (setq bottom-edge next-bottom-edge
+                            lowest-window w))) 'nomini)
     (select-window lowest-window)))
 
 (defconst appt-time-regexp
   "[0-9]?[0-9]\\(h\\([0-9][0-9]\\)?\\|[:.][0-9][0-9]\\)\\(am\\|pm\\)?")
 
 ;;;###autoload
-(defun appt-add (new-appt-time new-appt-msg)
-  "Add an appointment for today at NEW-APPT-TIME with message NEW-APPT-MSG.
-The time should be in either 24 hour format or am/pm format."
-  (interactive "sTime (hh:mm[am/pm]): \nsMessage: ")
-  (unless (string-match appt-time-regexp new-appt-time)
+(defun appt-add (time msg &optional warntime)
+  "Add an appointment for today at TIME with message MSG.
+The time should be in either 24 hour format or am/pm format.
+Optional argument WARNTIME is an integer (or string) giving the number
+of minutes before the appointment at which to start warning.
+The default is `appt-message-warning-time'."
+  (interactive "sTime (hh:mm[am/pm]): \nsMessage: 
+sMinutes before the appointment to start warning: ")
+  (unless (string-match appt-time-regexp time)
     (error "Unacceptable time-string"))
-  (let ((time-msg (list (list (appt-convert-time new-appt-time))
-                        (concat new-appt-time " " new-appt-msg) t)))
+  (and (stringp warntime)
+       (setq warntime (unless (string-equal warntime "")
+                        (string-to-number warntime))))
+  (and warntime
+       (not (integerp warntime))
+       (error "Argument WARNTIME must be an integer, or nil"))
+  (let ((time-msg (list (list (appt-convert-time time))
+                        (concat time " " msg) t)))
+    ;; It is presently non-sensical to have multiple warnings about
+    ;; the same appointment with just different delays, but it might
+    ;; not always be so.  TODO
+    (if warntime (setq time-msg (append time-msg (list warntime))))
     (unless (member time-msg appt-time-msg-list)
       (setq appt-time-msg-list
             (appt-sort-list (nconc appt-time-msg-list (list time-msg)))))))
@@ -513,27 +514,22 @@ The time should be in either 24 hour format or am/pm format."
   "Delete an appointment from the list of appointments."
   (interactive)
   (let ((tmp-msg-list appt-time-msg-list))
-    (while tmp-msg-list
-      (let* ((element (car tmp-msg-list))
-             (prompt-string (concat "Delete "
-                                   ;; We want to quote any doublequotes
-                                   ;; in the string, as well as put
-                                   ;; doublequotes around it.
-                                    (prin1-to-string
-                                    (substring-no-properties
-                                     (car (cdr element)) 0))
-                                    " from list? "))
-             (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)))))
-    (appt-check)
-    (message "")))
+    (dolist (element tmp-msg-list)
+      (if (y-or-n-p (concat "Delete "
+                            ;; We want to quote any doublequotes in the
+                            ;; string, as well as put doublequotes around it.
+                            (prin1-to-string
+                             (substring-no-properties (cadr element) 0))
+                            " from list? "))
+          (setq appt-time-msg-list (delq element appt-time-msg-list)))))
+  (appt-check)
+  (message ""))
 
 
 (defvar number)
 (defvar original-date)
 (defvar diary-entries-list)
+;; Autoload for the old way of using this package.  Can be removed sometime.
 ;;;###autoload
 (defun appt-make-list ()
   "Update the appointments list from today's diary buffer.
@@ -543,8 +539,7 @@ the function `appt-check').  We assume that the variables DATE and
 NUMBER hold the arguments that `diary-list-entries' received.
 They specify the range of dates that the diary is being processed for.
 
-Any appointments made with `appt-add' are not affected by this
-function.
+Any appointments made with `appt-add' are not affected by this function.
 
 For backwards compatibility, this function activates the
 appointment package (if it is not already active)."
@@ -569,14 +564,13 @@ appointment package (if it is not already active)."
               (setq appt-time-msg-list
                     (delq elt appt-time-msg-list))))
           (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.
-
+              ;; 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 ""))
+                    (new-time-string "")
+                    time-string)
                 ;; Skip diary entries for dates before today.
                 (while (and entry-list
                             (calendar-date-compare
@@ -585,50 +579,41 @@ appointment package (if it is not already active)."
                 ;; Parse the entries for today.
                 (while (and entry-list
                             (calendar-date-equal
-                             (calendar-current-date) (car (car entry-list))))
-                  (let ((time-string (cadr (car entry-list))))
-                    (while (string-match appt-time-regexp time-string)
-                      (let* ((beg (match-beginning 0))
-                             ;; Get just the time for this appointment.
-                             (only-time (match-string 0 time-string))
-                             ;; Find the end of this appointment
-                             ;; (the start of the next).
-                             (end (string-match
-                                   (concat "\n[ \t]*" appt-time-regexp)
-                                   time-string
-                                   (match-end 0)))
-                             ;; Get the whole string for this appointment.
-                             (appt-time-string
-                              (substring time-string beg (if end (1- end)))))
-
-                        ;; Add this appointment to appt-time-msg-list.
-                        (let* ((appt-time (list (appt-convert-time only-time)))
-                               (time-msg (list appt-time appt-time-string)))
-                          (setq appt-time-msg-list
-                                (nconc appt-time-msg-list (list time-msg))))
-
-                        ;; Discard this appointment from the string.
-                        (setq time-string
-                              (if end (substring time-string end) "")))))
+                             (calendar-current-date) (caar entry-list)))
+                  (setq time-string (cadr (car entry-list)))
+                  (while (string-match appt-time-regexp time-string)
+                    (let* ((beg (match-beginning 0))
+                           ;; Get just the time for this appointment.
+                           (only-time (match-string 0 time-string))
+                           ;; Find the end of this appointment
+                           ;; (the start of the next).
+                           (end (string-match
+                                 (concat "\n[ \t]*" appt-time-regexp)
+                                 time-string
+                                 (match-end 0)))
+                           ;; Get the whole string for this appointment.
+                           (appt-time-string
+                            (substring time-string beg end))
+                           (appt-time (list (appt-convert-time only-time)))
+                           (time-msg (list appt-time appt-time-string)))
+                      ;; Add this appointment to appt-time-msg-list.
+                      (setq appt-time-msg-list
+                            (nconc appt-time-msg-list (list time-msg))
+                            ;; Discard this appointment from the string.
+                            time-string
+                            (if end (substring time-string end) ""))))
                   (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.
-
+          ;; Convert current time to minutes after midnight (12:01am = 1),
+          ;; so that 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 (caar appt-time-msg-list))))
-
+                 (cur-comp-time (+ (* 60 (nth 2 now)) (nth 1 now)))
+                 (appt-comp-time (caar (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 (caar appt-time-msg-list))))))))))
+                  (setq appt-comp-time (caar (car appt-time-msg-list))))))))))
 
 
 (defun appt-sort-list (appt-list)
@@ -638,7 +623,7 @@ APPT-LIST is a list of the same format as `appt-time-msg-list'."
 
 
 (defun appt-convert-time (time2conv)
-  "Convert hour:min[am/pm] format to minutes from midnight.
+  "Convert hour:min[am/pm] format TIME2CONV to minutes from midnight.
 A period (.) can be used instead of a colon (:) to separate the
 hour and minute parts."
   ;; Formats that should be accepted:
@@ -649,27 +634,25 @@ hour and minute parts."
         (hr (if (string-match "[0-9]*[0-9]" time2conv)
                 (string-to-number (match-string 0 time2conv))
               0)))
-
-    ;; convert the time appointment time into 24 hour time
+    ;; Convert the time appointment time into 24 hour time.
     (cond ((and (string-match "pm" time2conv) (< hr 12))
-          (setq hr (+ 12 hr)))
-         ((and (string-match "am" time2conv) (= hr 12))
+           (setq hr (+ 12 hr)))
+          ((and (string-match "am" time2conv) (= hr 12))
            (setq hr 0)))
-
-    ;; convert the actual time into minutes.
+    ;; Convert the actual time into minutes.
     (+ (* hr 60) min)))
 
-
 (defun appt-update-list ()
   "If the current buffer is visiting the diary, update appointments.
-This function is intended for use with `write-file-functions'."
-  (and (string-equal buffer-file-name (expand-file-name diary-file))
+This function also acts on any file listed in `diary-included-files'.
+It is intended for use with `write-file-functions'."
+  (and (member buffer-file-name (append diary-included-files
+                                        (list (expand-file-name diary-file))))
        appt-timer
        (let ((appt-display-diary nil))
          (appt-check t)))
   nil)
 
-
 ;; In Emacs-21.3, the manual documented the following procedure to
 ;; activate this package:
 ;;     (display-time)
@@ -678,13 +661,13 @@ This function is intended for use with `write-file-functions'."
 ;; The display-time call was not necessary, AFAICS.
 ;; What was really needed was to add the hook and load this file.
 ;; Calling (diary 0) once the hook had been added was in some sense a
-;; roundabout way of loading this file. This file used to have code at
+;; roundabout way of loading this file.  This file used to have code at
 ;; the top-level that set up the appt-timer and global-mode-string.
 ;; One way to maintain backwards compatibility would be to call
-;; (appt-activate 1) at top-level. However, this goes against the
+;; (appt-activate 1) at top-level.  However, this goes against the
 ;; convention that just loading an Emacs package should not activate
-;; it. Instead, we make appt-make-list activate the package (after a
-;; suggestion from rms). This means that one has to call diary in
+;; it.  Instead, we make appt-make-list activate the package (after a
+;; suggestion from rms).  This means that one has to call diary in
 ;; order to get it to work, but that is in line with the old (weird,
 ;; IMO) documented behavior for activating the package.
 ;; Actually, since (diary 0) does not run diary-hook, I don't think
@@ -696,7 +679,7 @@ This function is intended for use with `write-file-functions'."
 
 ;;;###autoload
 (defun appt-activate (&optional arg)
-"Toggle checking of appointments.
+  "Toggle checking of appointments.
 With optional numeric argument ARG, turn appointment checking on if
 ARG is positive, otherwise off."
   (interactive "P")