unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#16868: Instrumenting macro for edebug turns it "not documented"
@ 2014-02-24 17:37 Juanma Barranquero
  2014-02-25 19:37 ` Glenn Morris
  0 siblings, 1 reply; 3+ messages in thread
From: Juanma Barranquero @ 2014-02-24 17:37 UTC (permalink / raw)
  To: 16868

Package: emacs
Severity: minor

Is there any reason that instrumenting a macro for edebug "loses" its
docstring, which does not happen for functions?

emacs -Q
eval the following code:

(defmacro test-macro (&rest body)
  "This is a test macro."
  `(progn ,@body))

C-h f test-macro <RET> => This is a test macro
C-u C-M-x
C-h f test-macro <RET> => Not documented.

I can see that it is nothing new, it already happens in 21.X. But it's puzzling.





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

* bug#16868: Instrumenting macro for edebug turns it "not documented"
  2014-02-24 17:37 bug#16868: Instrumenting macro for edebug turns it "not documented" Juanma Barranquero
@ 2014-02-25 19:37 ` Glenn Morris
  2014-02-25 20:49   ` Juanma Barranquero
  0 siblings, 1 reply; 3+ messages in thread
From: Glenn Morris @ 2014-02-25 19:37 UTC (permalink / raw)
  To: Juanma Barranquero; +Cc: 16868


Is it as simple as this?


*** lisp/emacs-lisp/edebug.el	2014-02-10 01:34:22 +0000
--- lisp/emacs-lisp/edebug.el	2014-02-25 19:35:39 +0000
***************
*** 1936,1946 ****
  	   [&optional stringp]
  	   [&optional ("interactive" interactive)]
  	   def-body))
- ;; FIXME? Isn't this missing the doc-string?  Cf defun.
  (def-edebug-spec defmacro
    ;; FIXME: Improve `declare' so we can Edebug gv-expander and
    ;; gv-setter declarations.
!   (&define name lambda-list [&optional ("declare" &rest sexp)] def-body))
  
  (def-edebug-spec arglist lambda-list)  ;; deprecated - use lambda-list.
  
--- 1936,1946 ----
  	   [&optional stringp]
  	   [&optional ("interactive" interactive)]
  	   def-body))
  (def-edebug-spec defmacro
    ;; FIXME: Improve `declare' so we can Edebug gv-expander and
    ;; gv-setter declarations.
!   (&define name lambda-list [&optional stringp]
!            [&optional ("declare" &rest sexp)] def-body))
  
  (def-edebug-spec arglist lambda-list)  ;; deprecated - use lambda-list.
  






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

* bug#16868: Instrumenting macro for edebug turns it "not documented"
  2014-02-25 19:37 ` Glenn Morris
@ 2014-02-25 20:49   ` Juanma Barranquero
  0 siblings, 0 replies; 3+ messages in thread
From: Juanma Barranquero @ 2014-02-25 20:49 UTC (permalink / raw)
  To: Glenn Morris; +Cc: 16868

On Tue, Feb 25, 2014 at 8:37 PM, Glenn Morris <rgm@gnu.org> wrote:

> Is it as simple as this?

Seems so. Thanks!





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

end of thread, other threads:[~2014-02-25 20:49 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-02-24 17:37 bug#16868: Instrumenting macro for edebug turns it "not documented" Juanma Barranquero
2014-02-25 19:37 ` Glenn Morris
2014-02-25 20:49   ` Juanma Barranquero

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).