Commit c175ad52 authored by Lars Ingebrigtsen's avatar Lars Ingebrigtsen
Browse files

Make dired-guess-default return all matching programs

* lisp/dired-x.el (dired-guess-default): Return all matching
programs (bug#48071).
parent 606b783a
Pipeline #11496 failed with stages
in 1 minute and 16 seconds
......@@ -972,38 +972,22 @@ REGEXP is matched case-sensitively."
(defun dired-guess-default (files)
"Return a shell command, or a list of commands, appropriate for FILES.
See `dired-guess-shell-alist-user'."
(let* ((case-fold-search dired-guess-shell-case-fold-search)
;; Prepend the user's alist to the default alist.
(alist (append dired-guess-shell-alist-user
(file (car files))
(flist (cdr files))
elt regexp cmds)
;; Find the first match in the alist for first file in FILES.
(while alist
(setq elt (car alist)
regexp (car elt)
alist (cdr alist))
(if (string-match-p regexp file)
(setq cmds (cdr elt)
alist nil)))
;; If more than one file, see if all of FILES match regular expression.
(while (and flist
(string-match-p regexp (car flist)))
(setq flist (cdr flist)))
;; If flist is still non-nil, then do not guess since this means that not
;; all the files in FILES were matched by the regexp.
(setq cmds (and (not flist) cmds))
;; Return commands or nil if flist is still non-nil.
;; Evaluate the commands in order that any logical testing will be done.
(if (cdr cmds)
(delete-dups (mapcar (lambda (cmd) (eval cmd `((file . ,file)))) cmds))
(eval (car cmds) `((file . ,file)))))) ; single command
(mapcar #'cdr
(seq-filter (lambda (elem)
(seq-some (lambda (file)
(string-match-p (car elem) file))
(append dired-guess-shell-alist-user
(if (length= programs 1)
(car programs)
(defun dired-guess-shell-command (prompt files)
"Ask user with PROMPT for a shell command, guessing a default from FILES."
......@@ -49,5 +49,17 @@
(sort (dired-get-marked-files 'local) #'string<))))
(delete-directory dir 'recursive))))
(ert-deftest dired-guess-default ()
(let ((dired-guess-shell-alist-user nil)
'(("\\.png\\'" "display")
("\\.gif\\'" "display" "xloadimage")
("\\.gif\\'" "feh")
("\\.jpe?g\\'" "xloadimage"))))
(should (equal (dired-guess-default '("/tmp/foo.png")) "display"))
(should (equal (dired-guess-default '("/tmp/foo.gif"))
'("display" "xloadimage" "feh")))))
(provide 'dired-x-tests)
;; dired-x-tests.el ends here
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment