unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Juanma Barranquero <lekktu@gmail.com>
Subject: CL function's docstrings
Date: Tue, 17 May 2005 01:52:06 +0200	[thread overview]
Message-ID: <f7ccd24b0505161652363001eb@mail.gmail.com> (raw)

Many cl*.el functions have a mismatch between the real argument names
and the ones used in their docstrings (mostly because many cl
functions do use some kind of old "standardized" naming scheme for
arguments, like cl-env, cl-keys, etc.).

For example, `cl-macroexpand' uses cl-macro and cl-env as arguments,
but FORM and ENVIRONMENT on the docstring. Unfortunately, this makes
`describe-function' less than useful for these functions (and defeats
argument highlighting).

AFAICS, there are four possible answers:

 1) Do nothing. The easier, if uglier, way :)

 2) Change the argument names to match the docstrings.
     Pros: it is the cleaner way.
     Cons: lot of change in the cl* files; less coherence between argument names
     among functions.

 3) Change the docstrings to match the argument names.
     Pro: easy.
     Cons: names are often much less descriptive than the ones already existing.

 4) Add \(fn ARG1 ARG2...) sections to docstrings.
     Pro: better info in `describe-function'; little change (one fn
line + one blank line
     per function's docstring, at most).
     Cons: many functions will need this, and \(fn ARG) forms are
uglier and more
     difficult to maintain.

I personally gravitate towards 2) or 4).

Does anyone else have an opinion about the issue?

-- 
                    /L/e/k/t/u

             reply	other threads:[~2005-05-16 23:52 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-05-16 23:52 Juanma Barranquero [this message]
2005-05-17 21:15 ` CL function's docstrings Stefan Monnier
2005-05-18 12:13   ` Kim F. Storm
2005-05-18 12:50     ` Juanma Barranquero
2005-05-18 22:45   ` Richard Stallman
2005-05-19 13:04     ` Stefan Monnier
2005-05-18  5:20 ` Richard Stallman
2005-05-18  8:09   ` Juanma Barranquero
2005-05-19  6:48     ` Richard Stallman
2005-05-19 23:57       ` Juanma Barranquero
2005-05-21  4:47         ` Richard Stallman

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=f7ccd24b0505161652363001eb@mail.gmail.com \
    --to=lekktu@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).