bug#11084: 24.0.94; [PATCH] Completion on DEF arg in grep-read-files is wrong.

[Thierry Volpiatto <thierry.volpiatto@gmail.com>]

Thierry Volpiatto <thierry.volpiatto@gmail.com> writes:

> Hi,
> it seem completion in grep-read-files is wrong for the default arg.
> It use (mapcar 'car grep-files-aliases) where it should use the cdr.
My bad, this is wrong the car should be used.
So here the patch:

--8<---------------cut here---------------start------------->8---
diff --git a/lisp/progmodes/grep.el b/lisp/progmodes/grep.el
--- a/lisp/progmodes/grep.el
+++ b/lisp/progmodes/grep.el
@@ -855,16 +855,17 @@
 	       default-extension
 	       (car grep-files-history)
 	       (car (car grep-files-aliases))))
+         (def-lst (delete-dups
+                   (delq nil (append (list default default-alias default-extension)
+                                     (mapcar 'car grep-files-aliases))))) 
 	 (files (completing-read
 		 (concat "Search for \"" regexp
 			 "\" in files"
 			 (if default (concat " (default " default ")"))
 			 ": ")
-		 'read-file-name-internal
+		 (append def-lst (all-completions "" 'read-file-name-internal))
 		 nil nil nil 'grep-files-history
-		 (delete-dups
-		  (delq nil (append (list default default-alias default-extension)
-				    (mapcar 'car grep-files-aliases)))))))
+                 default)))
     (and files
 	 (or (cdr (assoc files grep-files-aliases))
 	     files))))
--8<---------------cut here---------------end--------------->8---

Not using a list as DEF arg for completing-read is better.

-- 
  Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997