* help.el (help-for-help-internal): Don't purecopy the text (bug#4560).

* isearch.el (isearch-help-for-help-internal): Purecopy the second arg.
* help-macro.el (make-help-screen): Avoid using an ambiguous function
definition where the docstring could be taken for the return value.

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 9140b96cf8e0eb129002ef55a50069a1561aa1bf..9d12219081fdab87147d171e48484242df022df3 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,10 @@
+2009-09-27  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+       * help.el (help-for-help-internal): Don't purecopy the text (bug#4560).
+       * isearch.el (isearch-help-for-help-internal): Purecopy the second arg.
+       * help-macro.el (make-help-screen): Avoid using an ambiguous function
+       definition where the docstring could be taken for the return value.
+
 2009-09-26  Glenn Morris  <rgm@gnu.org>
 
        * mail/rmailmm.el (rmail-mime-show-images, rmail-mime-bulk-handler):

diff --git a/lisp/help-macro.el b/lisp/help-macro.el
index c920ecbf43b44c08b080113ec5900120d80b0cb6..802eb54916d011488c3a8d97c6aab19591b58ec7 100644 (file)
--- a/lisp/help-macro.el
+++ b/lisp/help-macro.el
@@ -90,7 +90,7 @@ When FNAME finally does get a command, it executes that command
 and then returns."
   (let ((doc-fn (intern (concat (symbol-name fname) "-doc"))))
     `(progn
-       (defun ,doc-fn () ,help-text)
+       (defun ,doc-fn () ,help-text nil)
        (defun ,fname ()
         "Help command."
         (interactive)

diff --git a/lisp/help.el b/lisp/help.el
index 25703fba9209c52889481ca33a9132ea0ce6032a..8b5efc88d8deded3375fb553d90e3076362a0fce 100644 (file)
--- a/lisp/help.el
+++ b/lisp/help.el
@@ -203,7 +203,9 @@ specifies what to do when the user exits the help buffer."
 ;; It can't find this, but nobody will look.
 (make-help-screen help-for-help-internal
   (purecopy "Type a help option: [abcCdefFgiIkKlLmnprstvw.] C-[cdefmnoptw] or ?")
-  (purecopy
+  ;; Don't purecopy this one, because it's not evaluated (it's
+  ;; directly used as a docstring in a function definition, so it'll
+  ;; be moved to the DOC file anyway: no need for purecopying it).
   "You have typed %THIS-KEY%, the help character.  Type a Help option:
 \(Use SPC or DEL to scroll through this text.  Type \\<help-map>\\[help-quit] to exit the Help command.)
 
@@ -248,7 +250,7 @@ C-n         News of recent Emacs changes.
 C-o         Emacs ordering and distribution information.
 C-p         Info about known Emacs problems.
 C-t         Emacs TODO list.
-C-w         Information on absence of warranty for GNU Emacs.")
+C-w         Information on absence of warranty for GNU Emacs."
   help-map)
 
 \f

diff --git a/lisp/isearch.el b/lisp/isearch.el
index 752746f09c287a04b9f152ad2ad3c434d67c0951..bf2f0738d977d56d2b2be643934d831e0c968b6a 100644 (file)
--- a/lisp/isearch.el
+++ b/lisp/isearch.el
@@ -347,7 +347,7 @@ A value of nil means highlight all matches."
 (eval-when-compile (require 'help-macro))
 
 (make-help-screen isearch-help-for-help-internal
-  "Type a help option: [bkm] or ?"
+  (purecopy "Type a help option: [bkm] or ?")
   "You have typed %THIS-KEY%, the help character.  Type a Help option:
 \(Type \\<help-map>\\[help-quit] to exit the Help command.)