unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH] lisp-mode: add docstring recognition for more common lisp symbols
@ 2023-02-09 10:39 Nicolas Martyanoff
  2023-02-09 14:12 ` Eli Zaretskii
  0 siblings, 1 reply; 4+ messages in thread
From: Nicolas Martyanoff @ 2023-02-09 10:39 UTC (permalink / raw)
  To: emacs-devel; +Cc: Nicolas Martyanoff

Add the 'doc-string-elt property to deftype, define-setf-expander and and
define-compiler-macro for proper docstring highlighting in Common Lisp.
---
 lisp/emacs-lisp/lisp-mode.el | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/lisp/emacs-lisp/lisp-mode.el b/lisp/emacs-lisp/lisp-mode.el
index 367f59e8785..d44c9d6e23d 100644
--- a/lisp/emacs-lisp/lisp-mode.el
+++ b/lisp/emacs-lisp/lisp-mode.el
@@ -181,8 +181,11 @@ lisp-mode-autoload-regexp
 (put 'define-category 'doc-string-elt 2)
 ;; CL
 (put 'defconstant 'doc-string-elt 3)
+(put 'define-compiler-macro 'doc-string-elt 3)
+(put 'define-setf-expander 'doc-string-elt 3)
 (put 'defparameter 'doc-string-elt 3)
 (put 'defstruct 'doc-string-elt 2)
+(put 'deftype 'doc-string-elt 3)
 
 (defvar lisp-doc-string-elt-property 'doc-string-elt
   "The symbol property that holds the docstring position info.")
-- 
2.39.1




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

* Re: [PATCH] lisp-mode: add docstring recognition for more common lisp symbols
  2023-02-09 10:39 [PATCH] lisp-mode: add docstring recognition for more common lisp symbols Nicolas Martyanoff
@ 2023-02-09 14:12 ` Eli Zaretskii
  2023-02-09 14:47   ` Nicolas Martyanoff
  0 siblings, 1 reply; 4+ messages in thread
From: Eli Zaretskii @ 2023-02-09 14:12 UTC (permalink / raw)
  To: Nicolas Martyanoff; +Cc: emacs-devel

> From: Nicolas Martyanoff <nicolas@n16f.net>
> Cc: Nicolas Martyanoff <nicolas@n16f.net>
> Date: Thu,  9 Feb 2023 11:39:14 +0100
> 
> Add the 'doc-string-elt property to deftype, define-setf-expander and and
> define-compiler-macro for proper docstring highlighting in Common Lisp.

Thanks, installed on the emacs-29 branch.

Please in the future try to follow our style for log messages more
closely.  As an example, look at how I modified your log message for
this commit.



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

* Re: [PATCH] lisp-mode: add docstring recognition for more common lisp symbols
  2023-02-09 14:12 ` Eli Zaretskii
@ 2023-02-09 14:47   ` Nicolas Martyanoff
  2023-02-09 15:48     ` Eli Zaretskii
  0 siblings, 1 reply; 4+ messages in thread
From: Nicolas Martyanoff @ 2023-02-09 14:47 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: Nicolas Martyanoff, emacs-devel

Eli Zaretskii <eliz@gnu.org> writes:

>> From: Nicolas Martyanoff <nicolas@n16f.net>
>> Cc: Nicolas Martyanoff <nicolas@n16f.net>
>> Date: Thu,  9 Feb 2023 11:39:14 +0100
>> 
>> Add the 'doc-string-elt property to deftype, define-setf-expander and and
>> define-compiler-macro for proper docstring highlighting in Common Lisp.
>
> Thanks, installed on the emacs-29 branch.
>
> Please in the future try to follow our style for log messages more
> closely.  As an example, look at how I modified your log message for
> this commit.

Ah you mean like some kind of file-by-file changelog? No problem.

Thank you for merging the patch!

Regards,

-- 
Nicolas Martyanoff
https://n16f.net
nicolas@n16f.net



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

* Re: [PATCH] lisp-mode: add docstring recognition for more common lisp symbols
  2023-02-09 14:47   ` Nicolas Martyanoff
@ 2023-02-09 15:48     ` Eli Zaretskii
  0 siblings, 0 replies; 4+ messages in thread
From: Eli Zaretskii @ 2023-02-09 15:48 UTC (permalink / raw)
  To: Nicolas Martyanoff; +Cc: emacs-devel

> From: Nicolas Martyanoff <nicolas@n16f.net>
> Cc: Nicolas Martyanoff <nicolas@n16f.net>,  emacs-devel@gnu.org
> Date: Thu, 09 Feb 2023 15:47:45 +0100
> 
> > Please in the future try to follow our style for log messages more
> > closely.  As an example, look at how I modified your log message for
> > this commit.
> 
> Ah you mean like some kind of file-by-file changelog? No problem.

Yes, we use ChangeLog-style log messages.  Of course, in addition to
the file names and function names, you can provide any supporting
free-style description of the changes, their reasons, and whatever
else you think is important.  But we want the file and function names
to be also present.

Thanks.



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

end of thread, other threads:[~2023-02-09 15:48 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-09 10:39 [PATCH] lisp-mode: add docstring recognition for more common lisp symbols Nicolas Martyanoff
2023-02-09 14:12 ` Eli Zaretskii
2023-02-09 14:47   ` Nicolas Martyanoff
2023-02-09 15:48     ` Eli Zaretskii

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