emacs-elpa-diffs
[Top][All Lists]
Advanced

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

[nongnu] elpa/helm b0d2bac7d4 1/5: Some more commentaries, no code chang


From: ELPA Syncer
Subject: [nongnu] elpa/helm b0d2bac7d4 1/5: Some more commentaries, no code change
Date: Tue, 20 Sep 2022 07:59:29 -0400 (EDT)

branch: elpa/helm
commit b0d2bac7d48a4ef07d3bb2438777c39b8bc0d225
Author: Thierry Volpiatto <thievol@posteo.net>
Commit: Thierry Volpiatto <thievol@posteo.net>

    Some more commentaries, no code change
---
 helm-files.el | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/helm-files.el b/helm-files.el
index b9c0b727f5..524a472398 100644
--- a/helm-files.el
+++ b/helm-files.el
@@ -3525,11 +3525,13 @@ in cache."
   "Return a callback for `file-notify-add-watch'."
   (lambda (event)
     (let ((desc (cadr event))
-          (target directory))
+          (target directory)) ; Either truename or directory.
       ;; `attribute-changed' means permissions have changed, not
       ;; file modifications like file changes, visit
       ;; etc... AFAIU the desc for this is `changed' and for our
-      ;; use case we don't care of this.
+      ;; use case we don't care of this. Elemnts of
+      ;; `helm-ff--list-directory-links' are of the form
+      ;; (truename . visited-symlink-directory)
       (when (memq desc '(created deleted renamed attribute-changed))
         ;; Watched directory is the truename which is not in the
         ;; cache, so remove its associated directory (the symlink)
@@ -3539,7 +3541,7 @@ in cache."
               (setq target (cdr it))
               (setq helm-ff--list-directory-links
                     (delete it helm-ff--list-directory-links))))
-        ;; When DIRECTORY is modified remove it from cache.
+        ;; When TARGET is modified remove it from cache.
         (remhash target helm-ff--list-directory-cache)
         ;; Remove watch as well in case of rename or delete.
         (file-notify-rm-watch (gethash target helm-ff--file-notify-watchers))



reply via email to

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