branch: elpa/dirvish
commit 4ed363024c8b826e92e5f92b114e4ebf50b5da24
Author: Alex Lu <hellosimon1...@hotmail.com>
Commit: Alex Lu <hellosimon1...@hotmail.com>

    refactor: WHY DOES EMACS31 OBSOLETE `when-let` AND `if-let`????????
---
 dirvish-tramp.el              |  2 +-
 dirvish.el                    | 28 ++++++++++++++--------------
 extensions/dirvish-emerge.el  |  8 ++++----
 extensions/dirvish-subtree.el |  4 ++--
 extensions/dirvish-yank.el    |  2 +-
 5 files changed, 22 insertions(+), 22 deletions(-)

diff --git a/dirvish-tramp.el b/dirvish-tramp.el
index 4c16f346f9..4cbafb22b5 100644
--- a/dirvish-tramp.el
+++ b/dirvish-tramp.el
@@ -102,7 +102,7 @@ FN is the original `dired-noselect' closure."
       (let ((process-connection-type nil)
             (localname (file-remote-p file 'localname))
             (buf (dirvish--util-buffer 'preview dv nil t)) proc)
-        (when-let ((proc (get-buffer-process buf))) (delete-process proc))
+        (when-let* ((proc (get-buffer-process buf))) (delete-process proc))
         (setq proc (start-file-process-shell-command
                     (buffer-name buf) buf
                     (format dirvish-tramp-preview-cmd localname localname)))
diff --git a/dirvish.el b/dirvish.el
index 34285637da..733243a86a 100644
--- a/dirvish.el
+++ b/dirvish.el
@@ -503,7 +503,7 @@ ARGS is a list of keyword arguments for `dirvish' struct."
           (setq header-line-format dirvish--header-line-fmt)))
       (cl-loop for (_d . b) in (dv-roots dv)
                when (not (eq b index)) do (kill-buffer b))
-      (when-let ((wconf (dv-winconf dv))) (set-window-configuration wconf)))
+      (when-let* ((wconf (dv-winconf dv))) (set-window-configuration wconf)))
     (mapc #'dirvish--kill-buffer (dv-preview-buffers dv))
     (cl-loop for b in (buffer-list) for bn = (buffer-name b) when
              (string-match-p (format " ?\\*Dirvish-.*-%s\\*" (dv-name dv)) bn)
@@ -674,7 +674,7 @@ buffer, it defaults to filename under the cursor when it is 
nil."
 
 (defun dirvish-thumb-buf-a (fn)
   "Advice for FN `image-dired-create-thumbnail-buffer'."
-  (when-let ((dv dirvish--this) ((dv-preview-window dv)))
+  (when-let* ((dv dirvish--this) ((dv-preview-window dv)))
     (dirvish--init-session dv)
     (with-selected-window (dv-preview-window dv)
       (switch-to-buffer image-dired-thumbnail-buffer)))
@@ -686,7 +686,7 @@ buffer, it defaults to filename under the cursor when it is 
nil."
     (with-current-buffer buf (add-hook 'post-command-hook fun nil t)) buf))
 
 (defun dirvish-dired-noselect-a (fn dir-or-list &optional flags)
-  "Return buffer for DIR with FLAGS, FN is `dired-noselect'."
+  "Return buffer for DIR-OR-LIST with FLAGS, FN is `dired-noselect'."
   (let* ((dir (if (consp dir-or-list) (car dir-or-list) dir-or-list))
          (key (file-name-as-directory (expand-file-name dir)))
          (this dirvish--this)
@@ -733,7 +733,7 @@ buffer, it defaults to filename under the cursor when it is 
nil."
 (defun dirvish-update-body-h (&optional force)
   "Update UI of current Dirvish.
 When FORCE, ensure the preview get refreshed."
-  (when-let ((dv (dirvish-curr)))
+  (when-let* ((dv (dirvish-curr)))
     (cond ((not dirvish-hide-cursor))
           ((eobp) (forward-line -1))
           ((cdr dired-subdir-alist))
@@ -741,7 +741,7 @@ When FORCE, ensure the preview get refreshed."
            (goto-char (dirvish-prop :content-begin))))
     (when dirvish-hide-cursor (dired-move-to-filename))
     (dirvish--render-attrs)
-    (when-let ((filename (dired-get-filename nil t)))
+    (when-let* ((filename (dired-get-filename nil t)))
       (dirvish-prop :index filename)
       (let ((h-buf (dirvish--util-buffer 'header dv t))
             (f-buf (dirvish--util-buffer 'footer dv t))
@@ -765,14 +765,14 @@ When FORCE, ensure the preview get refreshed."
 
 (defun dirvish-kill-buffer-h ()
   "Remove buffer from session's buffer list."
-  (when-let ((dv (dirvish-curr)) (buf (current-buffer)))
+  (when-let* ((dv (dirvish-curr)) (buf (current-buffer)))
     (let ((win (get-buffer-window buf)))
       (when (window-live-p win) (set-window-dedicated-p win nil)))
     (setf (dv-roots dv) (cl-remove-if (lambda (i) (eq (cdr i) buf)) (dv-roots 
dv)))
     (unless (dv-roots dv)
-      (when-let ((layout (car (dv-layout dv)))
-                 (wconf (dv-winconf dv))
-                 ((eq buf (window-buffer (selected-window)))))
+      (when-let* ((layout (car (dv-layout dv)))
+                  (wconf (dv-winconf dv))
+                  ((eq buf (window-buffer (selected-window)))))
         (set-window-configuration wconf))
       (remhash (dv-name dv) dirvish--session-hash)
       (cl-loop for b in (buffer-list) for bn = (buffer-name b) when
@@ -839,8 +839,8 @@ When FORCE, ensure the preview get refreshed."
 
 (dirvish-define-preview default (file ext)
   "Default preview dispatcher for FILE."
-  (when-let ((attrs (ignore-errors (file-attributes file)))
-             (size (file-attribute-size attrs)))
+  (when-let* ((attrs (ignore-errors (file-attributes file)))
+              (size (file-attribute-size attrs)))
     (cond ((file-directory-p file) ; default directory previewer
            (let* ((script `(with-current-buffer
                                (progn (setq insert-directory-program
@@ -873,7 +873,7 @@ When FORCE, ensure the preview get refreshed."
 (defun dirvish-shell-preview-proc-s (proc _exitcode)
   "A sentinel for dirvish preview process.
 When PROC finishes, fill preview buffer with process result."
-  (when-let ((dv (or (dirvish-curr) dirvish--this)))
+  (when-let* ((dv (or (dirvish-curr) dirvish--this)))
     (with-current-buffer (dirvish--util-buffer 'preview dv nil t)
       (erase-buffer) (remove-overlays)
       (insert (with-current-buffer (process-buffer proc) (buffer-string)))
@@ -890,7 +890,7 @@ When PROC finishes, fill preview buffer with process 
result."
 
 (defun dirvish--run-shell-for-preview (dv recipe)
   "Dispatch shell cmd with RECIPE for session DV."
-  (when-let ((proc (get-buffer-process (get-buffer " *Dirvish-temp*"))))
+  (when-let* ((proc (get-buffer-process (get-buffer " *Dirvish-temp*"))))
     (delete-process proc))
   (let ((buf (dirvish--util-buffer 'preview dv nil t))
         (proc (make-process :name "sh-out" :connection-type nil
@@ -1082,7 +1082,7 @@ LEVEL is the depth of current window."
          (depth (or (caar (dv-layout dv)) 0))
          (i 0))
     (dirvish--setup-mode-line (car (dv-layout dv)))
-    (when-let (fixed (nth 1 (dv-type dv))) (setq window-size-fixed fixed))
+    (when-let* ((fixed (nth 1 (dv-type dv)))) (setq window-size-fixed fixed))
     (set-window-dedicated-p
      nil (and (or (car (dv-layout dv)) (nth 2 (dv-type dv))) t))
     (set-window-fringes nil dirvish-window-fringe dirvish-window-fringe)
diff --git a/extensions/dirvish-emerge.el b/extensions/dirvish-emerge.el
index bf90cacb0c..9ca12eb1a1 100644
--- a/extensions/dirvish-emerge.el
+++ b/extensions/dirvish-emerge.el
@@ -108,7 +108,7 @@ turned on in the buffer."
 
 (cl-defmethod transient-infix-set ((obj dirvish-emerge-group) _value)
   "Set value for OBJ."
-  (if-let ((sel (oref obj selected)))
+  (if-let* ((sel (oref obj selected)))
       (dirvish-emerge-read-recipe (oref obj recipe) obj)
     (oset obj selected t)))
 
@@ -435,8 +435,8 @@ PREDS are locally composed predicates."
         (max-idx (length preds))
         (dir (file-local-name (dired-current-directory))))
     (while (< (point) end)
-      (when-let ((f-beg (dired-move-to-filename))
-                 (f-end (dired-move-to-end-of-filename)))
+      (when-let* ((f-beg (dired-move-to-filename))
+                  (f-end (dired-move-to-end-of-filename)))
         (let* ((l-beg (line-beginning-position))
                (l-end (1+ (line-end-position)))
                (local (buffer-substring-no-properties f-beg f-end))
@@ -458,7 +458,7 @@ PREDS are locally composed predicates."
                  (< (hash-table-count dirvish--attrs-hash)
                     dirvish-emerge-max-file-count)))
     (dirvish-emerge--readin-groups)
-    (when-let ((preds (dirvish-prop :emerge-preds)))
+    (when-let* ((preds (dirvish-prop :emerge-preds)))
       (dirvish-emerge--apply-1 preds))))
 
 ;;;; Interactive commands
diff --git a/extensions/dirvish-subtree.el b/extensions/dirvish-subtree.el
index b3aa612b25..970ea27536 100644
--- a/extensions/dirvish-subtree.el
+++ b/extensions/dirvish-subtree.el
@@ -328,7 +328,7 @@ See `dirvish-subtree-file-viewer' for details"
 (defun dirvish-subtree-up ()
   "Jump to beginning of current subtree."
   (interactive)
-  (when-let ((ov (dirvish-subtree--parent)))
+  (when-let* ((ov (dirvish-subtree--parent)))
     (goto-char (overlay-start ov))
     (dired-previous-line 1)))
 
@@ -397,7 +397,7 @@ This command takes a mouse event EV as its argument."
     (select-window win)
     (with-current-buffer (window-buffer win)
       (goto-char pos)
-      (when-let ((entry (dired-get-filename nil t)))
+      (when-let* ((entry (dired-get-filename nil t)))
         (if (file-directory-p entry)
             (dirvish-subtree-toggle)
           (dirvish-find-entry-a entry))))
diff --git a/extensions/dirvish-yank.el b/extensions/dirvish-yank.el
index 87b75c1e2c..38fe62f4f1 100644
--- a/extensions/dirvish-yank.el
+++ b/extensions/dirvish-yank.el
@@ -637,7 +637,7 @@ values."
   (if (and (slot-boundp obj 'value) (oref obj value))
       (oref obj value)
     ;; check saved values
-    (if-let ((saved (assq (oref obj command) transient-values)))
+    (if-let* ((saved (assq (oref obj command) transient-values)))
         (cdr saved)
       ;; use default value at last resort
       default-value)))

Reply via email to