changeset 5369:4141aeddc55b

Eliminate byte-compile warnings, core Lisp. 2011-03-10 Aidan Kehoe <kehoea@parhasard.net> * etags.el (buffer-tag-table-list): * files.el (find-file-read-only): * files.el (find-file-read-only-other-window): * info.el (Info-dir-outdated-p): * info.el (Info-dump-dir-entries): * info.el (Info-rebuild-dir): * menubar-items.el (default-menubar): * mouse.el (drag-window-divider): * mouse.el (vertical-divider-map): * test-harness.el (emacs-lisp-file-regexp): Eliminate byte-compile warnings, again aside from those linked to Stephen's various non-defined fontconfig functions.
author Aidan Kehoe <kehoea@parhasard.net>
date Thu, 10 Mar 2011 19:14:25 +0000
parents ed74d2ca7082
children 4c4b96b13f70
files lisp/ChangeLog lisp/etags.el lisp/files.el lisp/info.el lisp/menubar-items.el lisp/mouse.el lisp/test-harness.el
diffstat 7 files changed, 58 insertions(+), 41 deletions(-) [+]
line wrap: on
line diff
--- a/lisp/ChangeLog	Thu Mar 10 18:51:15 2011 +0000
+++ b/lisp/ChangeLog	Thu Mar 10 19:14:25 2011 +0000
@@ -1,3 +1,18 @@
+2011-03-10  Aidan Kehoe  <kehoea@parhasard.net>
+
+	* etags.el (buffer-tag-table-list):
+	* files.el (find-file-read-only):
+	* files.el (find-file-read-only-other-window):
+	* info.el (Info-dir-outdated-p):
+	* info.el (Info-dump-dir-entries):
+	* info.el (Info-rebuild-dir):
+	* menubar-items.el (default-menubar):
+	* mouse.el (drag-window-divider):
+	* mouse.el (vertical-divider-map):
+	* test-harness.el (emacs-lisp-file-regexp):
+	Eliminate byte-compile warnings, again aside from those linked to
+	Stephen's various non-defined fontconfig functions.
+
 2011-03-10  Aidan Kehoe  <kehoea@parhasard.net>
 
 	* cmdloop.el (yes-or-no-p):
--- a/lisp/etags.el	Thu Mar 10 18:51:15 2011 +0000
+++ b/lisp/etags.el	Thu Mar 10 19:14:25 2011 +0000
@@ -250,8 +250,8 @@
 	     (and (file-readable-p name)
 		  ;; get-tag-table-buffer has side-effects
 		  (list (symbol-value-in-buffer 'buffer-file-name
-						(get-tag-table-buffer name))))))
-	   result)
+						(get-tag-table-buffer name)))))
+	   result))
     ;; If no TAGS file has been found, ask the user explicitly.
     ;; #### tags-file-name is *evil*.
     (or result tags-file-name
--- a/lisp/files.el	Thu Mar 10 18:51:15 2011 +0000
+++ b/lisp/files.el	Thu Mar 10 19:14:25 2011 +0000
@@ -998,9 +998,9 @@
 			  (read-coding-system "Coding system: "))
 		     t))
   (let ((value (find-file filename codesys wildcards)))
-    (mapcar #'(lambda (buffer)
-		(set-symbol-value-in-buffer 'buffer-read-only t buffer))
-	    (if (listp value) value (list value)))
+    (mapc #'(lambda (buffer)
+              (set-symbol-value-in-buffer 'buffer-read-only t buffer))
+          (if (listp value) value (list value)))
     value))
 
 (defun find-file-read-only-other-window (filename &optional codesys wildcards)
--- a/lisp/info.el	Thu Mar 10 18:51:15 2011 +0000
+++ b/lisp/info.el	Thu Mar 10 19:14:25 2011 +0000
@@ -1127,7 +1127,7 @@
   (let ((dir-mod-time (nth 5 (file-attributes file)))
 	f-mod-time newer)
     (setq Info-dir-newer-info-files nil)
-    (mapcar
+    (mapc
      #'(lambda (f)
 	 (prog2
 	     (setq f-mod-time (nth 5 (file-attributes f)))
@@ -1191,23 +1191,23 @@
   (let ((tab-width 8)
 	(description-col 0)
 	len)
-    (mapcar #'(lambda (e)
-		(setq e (cdr e))	; Drop filename
-		(setq len (length (concat (car e)
-					  (car (cdr e)))))
-		(if (> len description-col)
-		    (setq description-col len)))
-	    entries)
+    (mapc #'(lambda (e)
+              (setq e (cdr e))	; Drop filename
+              (setq len (length (concat (car e)
+                                        (car (cdr e)))))
+              (if (> len description-col)
+                  (setq description-col len)))
+          entries)
     (setq description-col (+ 5 description-col))
-    (mapcar #'(lambda (e)
-		(setq e (cdr e))	; Drop filename
-		(insert "* " (car e) ":" (car (cdr e)))
-		(setq e (car (cdr (cdr e))))
-		(while e
-		  (indent-to-column description-col)
-		  (insert (car e) "\n")
-		  (setq e (cdr e))))
-	    entries)
+    (mapc #'(lambda (e)
+              (setq e (cdr e))	; Drop filename
+              (insert "* " (car e) ":" (car (cdr e)))
+              (setq e (car (cdr (cdr e))))
+              (while e
+                (indent-to-column description-col)
+                (insert (car e) "\n")
+                (setq e (cdr e))))
+          entries)
     (insert "\n")))
 
 
@@ -1299,7 +1299,7 @@
 	    (narrow-to-region mark next-section)
 	    (setq dir-section-contents (nreverse (Info-parse-dir-entries
 						  (point-min) (point-max))))
-	    (mapcar
+	    (mapc
 	     #'(lambda (file)
 		 (setq dir-entry (assoc (downcase
 					 (file-name-sans-extension
--- a/lisp/menubar-items.el	Thu Mar 10 18:51:15 2011 +0000
+++ b/lisp/menubar-items.el	Thu Mar 10 19:14:25 2011 +0000
@@ -262,7 +262,7 @@
 		    (submenu-generate-accelerator-spec
 		     (mapcar #'(lambda (bmk)
 				 `[,bmk (bookmark-delete ',bmk)])
-			     (bookmark-all-names)))))
+			     (declare-fboundp (bookmark-all-names))))))
       ["%_Edit Bookmark List" bookmark-bmenu-list
        :active (and-boundp 'bookmark-alist bookmark-alist)]
       "---"
--- a/lisp/mouse.el	Thu Mar 10 18:51:15 2011 +0000
+++ b/lisp/mouse.el	Thu Mar 10 19:14:25 2011 +0000
@@ -1781,22 +1781,24 @@
 			 (let ((all-that-bad nil)
 			       (new-left-ok nil)
 			       (new-right-ok nil))
-			   (mapcar* (lambda (window old-edges)
-				      (let ((new (car (window-pixel-edges window))))
-					(if (/= new (car old-edges))
-					    (if (and new-left-ok
-						     (/= new-left-ok new))
-						(setq all-that-bad t)
-					      (setq new-left-ok new)))))
-				    (window-list) old-edges-all-windows)
-			   (mapcar* (lambda (window old-edges)
-				      (let ((new (caddr (window-pixel-edges window))))
-					(if (/= new (caddr old-edges))
-					    (if (and new-right-ok
-						     (/= new-right-ok new))
-						(setq all-that-bad t)
-					      (setq new-right-ok new)))))
-				    (window-list) old-edges-all-windows)
+			   (mapc (lambda (window old-edges)
+                                   (let ((new
+                                          (car (window-pixel-edges window))))
+                                     (if (/= new (car old-edges))
+                                         (if (and new-left-ok
+                                                  (/= new-left-ok new))
+                                             (setq all-that-bad t)
+                                           (setq new-left-ok new)))))
+                                 (window-list) old-edges-all-windows)
+			   (mapc (lambda (window old-edges)
+                                   (let ((new
+                                          (caddr (window-pixel-edges window))))
+                                     (if (/= new (caddr old-edges))
+                                         (if (and new-right-ok
+                                                  (/= new-right-ok new))
+                                             (setq all-that-bad t)
+                                           (setq new-right-ok new)))))
+                                 (window-list) old-edges-all-windows)
 			   all-that-bad))
 		     (set-window-configuration backup-conf)))))))))
 
--- a/lisp/test-harness.el	Thu Mar 10 18:51:15 2011 +0000
+++ b/lisp/test-harness.el	Thu Mar 10 19:14:25 2011 +0000
@@ -128,7 +128,7 @@
 
 (defvar test-harness-current-file nil)
 
-(defvar emacs-lisp-file-regexp (purecopy "\\.el\\'")
+(defvar emacs-lisp-file-regexp "\\.el\\'"
   "*Regexp which matches Emacs Lisp source files.")
 
 (defconst test-harness-file-summary-template