* recentf.el (recentf-cleanup): Use a hash table to find

duplicates (Bug#4407).

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 11bfa9bce85b6ceceb94632aa97eb9f0266e1043..0ad8e2f07cf89e0b235bda416a7eb648d237d445 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,8 @@
+2009-09-13  Vincent Belaïche  <vincent.belaiche@gmail.com>
+
+       * recentf.el (recentf-cleanup): Use a hash table to find
+       duplicates (Bug#4407).
+
 2009-09-13  Per Starbäck  <per@starback.se>  (tiny change)
 
        * textmodes/ispell.el (ispell-command-loop): Convert keys such as

diff --git a/lisp/recentf.el b/lisp/recentf.el
index bc8904f92112e0165007927dc896ace2a2673de7..c0fa933840af526aea67b66cbf32f40f212d935b 100644 (file)
--- a/lisp/recentf.el
+++ b/lisp/recentf.el
@@ -1307,13 +1307,20 @@ empty `file-name-history' with the recent list."
 That is, remove duplicates, non-kept, and excluded files."
   (interactive)
   (message "Cleaning up the recentf list...")
-  (let ((n 0) newlist)
+  (let ((n 0)
+       (ht (make-hash-table
+            :size recentf-max-saved-items
+            :test 'equal))
+       newlist key)
     (dolist (f recentf-list)
-      (setq f (recentf-expand-file-name f))
+      (setq f (recentf-expand-file-name f)
+           key (if recentf-case-fold-search (downcase f) f))
       (if (and (recentf-include-p f)
                (recentf-keep-p f)
-               (not (recentf-string-member f newlist)))
-          (push f newlist)
+               (not (gethash key ht)))
+         (progn
+           (push f newlist)
+           (puthash key t ht))
         (setq n (1+ n))
         (message "File %s removed from the recentf list" f)))
     (message "Cleaning up the recentf list...done (%d removed)" n)