Re: [PATCH] Add file-dwim.

[Artur Malabarba]

> +  :group 'file-dwim)

These :group entries are redundant. It is a (poorly documented)
feature that all defcustoms following a defgroup belong to that
defgroup.

> +;; (defun play-video-file (file)
> +;;   (shell-command (concat "my-video-player " (shell-quote-argument file))))

Why not say that the cdr of an entry in `file-dwim-action-list' can
also be a list like ("command" "arg1" "arg2"), in which case it used
in `start-process' with the file-name at the end?
For instance:

    (add-to-list 'file-dwim-action-list
                 (list (rx "." (or "mp3" "flac" "aac" "wav") eos) "mplayer"))

I think that would be very valuable, as this sounds like the common
use for this package.