(appt-select-lowest-window): Use walk-windows

instead of cycling through windows with next-window.

diff --git a/lisp/calendar/appt.el b/lisp/calendar/appt.el
index 04c8a3dd6079ddf01e38f71df8f16650abcf11d0..623d94cd96a21abbae690eac0dc2f517a2188a99 100644 (file)
--- a/lisp/calendar/appt.el
+++ b/lisp/calendar/appt.el
@@ -422,24 +422,14 @@ Usually just deletes the appointment buffer."
 
 (defun appt-select-lowest-window ()
 "Select the lowest window on the frame."
-  (let* ((lowest-window (selected-window))
-        (bottom-edge (nth 3 (window-edges)))
-         (last-window (previous-window))
-         (window-search t))
-    (while window-search
-      (let* ((this-window (next-window))
-             (next-bottom-edge (nth 3 (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)))))))
-
+  (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)