unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#19685: [PATCH] define-minor-mode docstring generation bug
@ 2015-01-25  8:52 Kelly Dean
  2015-02-18  1:04 ` bug#19685: " Kelly Dean
  0 siblings, 1 reply; 2+ messages in thread
From: Kelly Dean @ 2015-01-25  8:52 UTC (permalink / raw)
  To: 19685

[-- Attachment #1: Type: text/plain, Size: 343 bytes --]

Try this:
(define-minor-mode foo "foo" :global t)

The variable it creates includes a docstring with

⌜Setting this variable directly does not take effect⌝.

That's a bug.

The attached dmm-docstring-gen-bug.patch fixes it.

Also attached is dmm-docstring-clarification.patch. (Not a bug, but might as well include it here.)


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: dmm-docstring-gen-bug.patch --]
[-- Type: text/x-diff, Size: 475 bytes --]

--- emacs-24.4/lisp/emacs-lisp/easy-mmode.el
+++ emacs-24.4/lisp/emacs-lisp/easy-mmode.el
@@ -158,7 +158,8 @@
   ;; Allow skipping the first three args.
   (cond
    ((keywordp init-value)
-    (setq body `(,init-value ,lighter ,keymap ,@body)
+    (setq body (if keymap `(,init-value ,lighter ,keymap ,@body)
+		 `(,init-value ,lighter))
 	  init-value nil lighter nil keymap nil))
    ((keywordp lighter)
     (setq body `(,lighter ,keymap ,@body) lighter nil keymap nil))

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #3: dmm-docstring-clarification.patch --]
[-- Type: text/x-diff, Size: 962 bytes --]

--- emacs-24.4/lisp/emacs-lisp/easy-mmode.el
+++ emacs-24.4/lisp/emacs-lisp/easy-mmode.el
@@ -114,9 +114,11 @@
 BODY contains code to execute each time the mode is enabled or disabled.
   It is executed after toggling the mode, and before running MODE-hook.
   Before the actual body code, you can write keyword arguments, i.e.
-  alternating keywords and values.  These following special keywords
-  are supported (other keywords are passed to `defcustom' if the minor
-  mode is global):
+  alternating keywords and values.  If you provide BODY, then you must
+  provide INIT-VALUE, LIGHTER, and KEYMAP, or provide at least one
+  keyword argument, or both; otherwise, BODY would be misinterpreted.
+  The following special keywords are supported (other keywords are passed
+  to `defcustom' if the minor mode is global):
 
 :group GROUP	Custom group name to use in all generated `defcustom' forms.
 		Defaults to MODE without the possible trailing \"-mode\".

^ permalink raw reply	[flat|nested] 2+ messages in thread

* bug#19685: define-minor-mode docstring generation bug
  2015-01-25  8:52 bug#19685: [PATCH] define-minor-mode docstring generation bug Kelly Dean
@ 2015-02-18  1:04 ` Kelly Dean
  0 siblings, 0 replies; 2+ messages in thread
From: Kelly Dean @ 2015-02-18  1:04 UTC (permalink / raw)
  To: 19685-done

Fixed in trunk.





^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-02-18  1:04 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-01-25  8:52 bug#19685: [PATCH] define-minor-mode docstring generation bug Kelly Dean
2015-02-18  1:04 ` bug#19685: " Kelly Dean

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).