[Top][All Lists]

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

[patch] 21.3 executable.find - Use cache in `executable-find'

From: Jari Aalto+mail.linux
Subject: [patch] 21.3 executable.find - Use cache in `executable-find'
Date: Sun, 08 Feb 2004 13:37:16 +0200
User-agent: Gnus/5.110002 (No Gnus v0.2) Emacs/21.3 (windows-nt) (i386-msvc-nt5.0.2195)

It is common that searches for programs are done multiple times, so it
would be good if the precious values were cached just like bash does.
Here is patch to make this happen.

This patch supposes that my other patches have been applied
(See patch to implement `executable-command-find-unix-p')

2004-02-08 Sun  Jari Aalto  <jari.aalto <AT> poboxes.com>

        * progmodes/executable.el (executable-find-cache): New variable.
        (executable-find-sans-suffix): New function.
        (executable-find-path): Renamed. Was `executable-find'.
        (executable-find-cache): New function.
        (executable-find): Rewritten. Now utilizes cache
        `executable-find-cache'. Added new OPTIONAL parameter `no-cache'
        to force searching COMMAND again.

Index: executable.el
RCS file: 
retrieving revision 1.5
retrieving revision 1.6
diff -u -IId: -u -b -w -r1.5 -r1.6
--- executable.el       3 Feb 2004 16:34:18 -0000       1.5
+++ executable.el       8 Feb 2004 11:30:11 -0000       1.6
@@ -107,6 +107,10 @@
 (defvar executable-command nil)
+(defvar executable-find-cache nil
+  "*`executable-find' cache.
+Syntax: '((program path) ...).")
 (defcustom executable-self-display "tail"
   "*Command you use with argument `+2' to make text files self-display.
 Note that the like of `more' doesn't work too well under Emacs \\[shell]."
@@ -205,10 +209,21 @@
          ((search-forward "find: bad option -maxdepth" nil t)
+(defun executable-find-sans-suffix (command)
+  "Return COMMAND without `executable-binary-suffixes'."
+  (let ((ext (file-name-extension command)))
+    (if (and ext
+             (member (concat "." ext) executable-binary-suffixes))
+        (file-name-sans-extension command)
+      command)))
+(defsubst executable-find-cache (command)
+  "Look up COMMAND from `executable-find-cache'."
+  (assoc command executable-find-cache))
-(defun executable-find (command)
-  "Search for COMMAND in exec-path and return the absolute file name.
-Return nil if COMMAND is not found anywhere in `exec-path'."
+(defun executable-find-path (command)
+  "Search for COMMAND in exec-path and return the absolute file name."
   (let ((list exec-path)
     (while list
@@ -229,6 +244,29 @@
               (cdr list))))
+(defun executable-find (command &optional no-cache)
+  "Search for COMMAND in exec-path and return the absolute file name.
+Return nil if COMMAND is not found anywhere in `exec-path'.
+Optional parameter NO-CACHE says to search all PATH hierarchy
+without looking the program location from `executable-find-cache'."
+  (let (cache
+        path)
+    (unless no-cache
+      (setq path (executable-find-cache
+                  (executable-find-sans-suffix command))))
+    (unless path
+      ;;  Was not in cache, or user requested full PATH search
+      (when (setq path (executable-find-path command))
+        ;;  Record this to cache
+        (setq cache
+              (list (executable-find-sans-suffix
+                     (file-name-nondirectory command))
+                    path))))
+    (when cache
+      (push cache executable-find-cache))
+    path))
 (defun executable-chmod ()
   "This gets called after saving a file to assure that it be executable.
 You can set the absolute or relative mode in variable `executable-chmod' for
@@ -241,7 +279,6 @@
                                (- executable-chmod)
                              (logior executable-chmod
                                      (file-modes buffer-file-name)))))))
 (defun executable-interpret (command)
   "Run script with user-specified args, and collect output in a buffer.

Swatch @time   http://www.mir.com.my/iTime/itime.htm
Use Licenses!  http://www.linuxjournal.com/article.php?sid=6225
Which Licence? http://www.linuxjournal.com/article.php?sid=4825
OSI Licences   http://www.opensource.org/licenses/

reply via email to

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