all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Hong Xu <hong@topbug.net>
To: Eli Zaretskii <eliz@gnu.org>
Cc: larsi@gnus.org, 37538@debbugs.gnu.org
Subject: bug#37538: [PATCH] Add docstring for `tags-complete-tags-table-file'.
Date: Wed, 9 Oct 2019 15:58:46 -0700	[thread overview]
Message-ID: <c73d1ade-2af1-f449-2c21-a7a39d1b2375@topbug.net> (raw)
In-Reply-To: <83a7aawdar.fsf@gnu.org>

On 10/9/19 1:09 AM, Eli Zaretskii wrote:
>> Cc: Eli Zaretskii <eliz@gnu.org>, 37538@debbugs.gnu.org
>> From: Hong Xu <hong@topbug.net>
>> Date: Tue, 8 Oct 2019 23:39:08 -0700
>>
>> On 10/8/19 9:21 AM, Lars Ingebrigtsen wrote:
>>>
>>> The problem is that `(elisp) Programmed Completion' (at least in Emacs
>>> 27) doesn't mention a WHAT parameter at all, so it's unclear what this
>>> refers to.
>>
>> How about the following change?
> 
> I don't see how it solves the problem, sorry.  It attempts to assign
> some significance to the ordinal number of the argument WHAT, in the
> hope that the reader will be able to connect the dots.  But a good
> documentation should not leave the dots unconnected, it should spell
> them out, ideally in one place.
> 
> I actually don't understand why we would like to send the reader to
> the manual, instead of describing the effects of WHAT in the doc
> string.  Can we just say it right there?
> 

The description was quite long and I don't think it is justifiable to copy so much text over here to the docstring, plus there are additional reference in the referred manual section.

IMO the docstring here really tells the reader that this is a standard completion function -- I doubt anyone would consult this docstring to call `tags-complete-tags-table-file', but most likely want to understand that this is a completion function so that they can better understand the codebase. To read the referred portion of the manual, for the purpose of understanding the codebase, is a somewhat inevitable task by itself.






  reply	other threads:[~2019-10-09 22:58 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-28  7:59 bug#37538: [PATCH] Add docstring for `tags-complete-tags-table-file' Hong Xu
2019-09-28  8:11 ` Eli Zaretskii
2019-09-28 18:40   ` Hong Xu
2019-10-07  4:13     ` Lars Ingebrigtsen
2019-10-07 16:18       ` Eli Zaretskii
2019-10-07 17:27         ` Hong Xu
2019-10-08 16:21           ` Lars Ingebrigtsen
2019-10-09  6:39             ` Hong Xu
2019-10-09  8:09               ` Eli Zaretskii
2019-10-09 22:58                 ` Hong Xu [this message]
2019-10-10  7:49                   ` Eli Zaretskii
2019-10-12  0:48                     ` Hong Xu
2019-10-12  9:04                       ` Eli Zaretskii
2019-10-12 18:46                         ` Hong Xu
2019-10-12 18:57                           ` Eli Zaretskii

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

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

  git send-email \
    --in-reply-to=c73d1ade-2af1-f449-2c21-a7a39d1b2375@topbug.net \
    --to=hong@topbug.net \
    --cc=37538@debbugs.gnu.org \
    --cc=eliz@gnu.org \
    --cc=larsi@gnus.org \
    /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 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.