all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Dmitry Gutov <dgutov@yandex.ru>
Cc: emacs-devel@gnu.org
Subject: Re: [Emacs-diffs] master 2475687: Improve documentation changes of a recent commit
Date: Sun, 14 Apr 2019 17:40:28 +0300	[thread overview]
Message-ID: <835zrgk5ur.fsf@gnu.org> (raw)
In-Reply-To: <f4b79118-8189-d260-7051-a0c9fa1eac02@yandex.ru> (message from Dmitry Gutov on Sun, 14 Apr 2019 13:34:51 +0300)

> From: Dmitry Gutov <dgutov@yandex.ru>
> Date: Sun, 14 Apr 2019 13:34:51 +0300
> 
> FTR, I quite dislike this kind of duplication in docstrings.

Yes, you've said that in the past, and I think we agreed to disagree
on it.  My opinion is that it's a judgment call: sometimes duplication
is for the better, and sometimes for the worse.  In this case, the
original doc string of json-parse-buffer said almost nothing useful,
leaving almost everything to look up in another function's
documentation, which IME is annoying.  Referring to another doc string
is OK for some secondary details, or perhaps for some very complicated
issues.  Not for the main part of the doc.  Especially for a function
whose name doesn't necessarily speak volumes about its purpose.

> Didn't we discuss the difference between docstrings and manuals some 
> time ago, where you expressed the opinion that docstrings are allowed 
> the kind of "see XX for more detail" references, and it's the manuals 
> where information sometimes has to be reiterated for the convenience of 
> the reader?

Doesn't sound like something I'd say, not to that effect.  "Allowed",
yes; but not "required".  If anything, it is easier to refer to a
previous function in the manual, when two or more functions are
described one after another.  By contrast, doc strings are never
"near" one another.

> Here you seem to have made the reverse choice.

I didn't like the original doc string, yes.  The manual I didn't
change, it was written that way to begin with.



  reply	other threads:[~2019-04-14 14:40 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20190413070933.31645.83730@vcs0.savannah.gnu.org>
     [not found] ` <20190413070934.DDF7B202C6@vcs0.savannah.gnu.org>
2019-04-14 10:34   ` [Emacs-diffs] master 2475687: Improve documentation changes of a recent commit Dmitry Gutov
2019-04-14 14:40     ` Eli Zaretskii [this message]
2019-04-14 20:34       ` Dmitry Gutov
2019-04-15 14:57         ` Eli Zaretskii
2019-04-15 15:42           ` Dmitry Gutov
2019-04-15 16:06             ` Eli Zaretskii
2019-04-15 17:15               ` Dmitry Gutov

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=835zrgk5ur.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=dgutov@yandex.ru \
    --cc=emacs-devel@gnu.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.