all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#66828: [PATCH] Add docstrings in cl-lib.el
@ 2023-10-29 22:07 Jeremy Bryant via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2023-10-30 11:43 ` Eli Zaretskii
  0 siblings, 1 reply; 2+ messages in thread
From: Jeremy Bryant via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2023-10-29 22:07 UTC (permalink / raw)
  To: 66828

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

Tags: patch

Hello Emacs maintainers,

Following recent discussions on documentation in cl-lib, I would like to
submit a simple patch, creating docstrings where there were none
previously.

Feedback welcome on format, conventions and any other aspects.

Best,
Jeremy



[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-Add-docstrings-in-cl-lib.el.patch --]
[-- Type: text/patch, Size: 1141 bytes --]

From 02e487816f5542d30b8cdf0e3f748926f9b7cf2e Mon Sep 17 00:00:00 2001
From: Jeremy Bryant <jb@jeremybryant.net>
Date: Sun, 29 Oct 2023 21:56:54 +0000
Subject: [PATCH] Add docstrings in cl-lib.el

* lisp/emacs-lisp/cl-lib.el (cl--set-buffer-substring): Add docstring

(cl--defalias): Add docstring
---
 lisp/emacs-lisp/cl-lib.el | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/lisp/emacs-lisp/cl-lib.el b/lisp/emacs-lisp/cl-lib.el
index f4c1ac85b13..29b6c2f3545 100644
--- a/lisp/emacs-lisp/cl-lib.el
+++ b/lisp/emacs-lisp/cl-lib.el
@@ -157,6 +157,7 @@ cl-pushnew
     `(cl-callf2 cl-adjoin ,x ,place ,@keys)))
 
 (defun cl--set-buffer-substring (start end val)
+  "Delete region from START to END and insert VAL."
   (save-excursion (delete-region start end)
 		  (goto-char start)
 		  (insert val)
@@ -194,6 +195,9 @@ 'cl--block-throw
 ;; the target form to return the values as a list.
 
 (defun cl--defalias (cl-f el-f &optional doc)
+  "Define function CL-F as definition EL-F.
+
+For example, (cl--defalias 'cl-first 'car)."
   (defalias cl-f el-f doc)
   (put cl-f 'byte-optimizer 'byte-compile-inline-expand))
 
-- 
2.40.1


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

* bug#66828: [PATCH] Add docstrings in cl-lib.el
  2023-10-29 22:07 bug#66828: [PATCH] Add docstrings in cl-lib.el Jeremy Bryant via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2023-10-30 11:43 ` Eli Zaretskii
  0 siblings, 0 replies; 2+ messages in thread
From: Eli Zaretskii @ 2023-10-30 11:43 UTC (permalink / raw)
  To: Jeremy Bryant; +Cc: 66828-done

> Date: Sun, 29 Oct 2023 22:07:13 +0000
> From:  Jeremy Bryant via "Bug reports for GNU Emacs,
>  the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>
> 
> Following recent discussions on documentation in cl-lib, I would like to
> submit a simple patch, creating docstrings where there were none
> previously.

Thanks, installed on the emacs-29 branch.

> Feedback welcome on format, conventions and any other aspects.

About this: see how I modified the commit log message.





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

end of thread, other threads:[~2023-10-30 11:43 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-10-29 22:07 bug#66828: [PATCH] Add docstrings in cl-lib.el Jeremy Bryant via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-10-30 11:43 ` Eli Zaretskii

Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.