all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Ram Bhamidipaty <rbhamidipaty@roku.com>
To: John Wiegley <jwiegley@gmail.com>, Eli Zaretskii <eliz@gnu.org>
Cc: "22220@debbugs.gnu.org" <22220@debbugs.gnu.org>
Subject: bug#22220: 24.5; no documentation for default process sentinel
Date: Wed, 23 Dec 2015 17:54:12 +0000	[thread overview]
Message-ID: <BLUPR0101MB1553E8B64A52BA58870DE997BAE60@BLUPR0101MB1553.prod.exchangelabs.com> (raw)
In-Reply-To: <m2k2o59ii6.fsf@newartisans.com>

There is also a slight problem with the set-process-sentinel docs - there really isn't anything that says a process starts with a default sentinel - even if the set-process-sentinel function is not called.

The current docs set-process-sentinel docs just say what happens if the sentinel argument is nil. Careful reading certainly implies that the default sentinel will be given to a process - but surely this kind of information can be more explicit.

-Ram

-----Original Message-----
From: John Wiegley [mailto:jwiegley@gmail.com] 
Sent: Wednesday, December 23, 2015 9:34 AM
To: Eli Zaretskii <eliz@gnu.org>
Cc: Ram Bhamidipaty <rbhamidipaty@roku.com>; 22220@debbugs.gnu.org
Subject: Re: bug#22220: 24.5; no documentation for default process sentinel

>>>>> Eli Zaretskii <eliz@gnu.org> writes:

>> It would be nice if the default process sentinel were explicitly documented
>> - at least the name and formal description.

> As for the name, why is that important? You can always get it installed for
> the process using nil as the argument to set-process-sentinel, and the
> implementation is in C, so how would the name help?

I must admit to a similar confusion. Aside from the fact that more could be
said: does more need to be said?

-- 
John Wiegley                  GPG fingerprint = 4710 CF98 AF9B 327B B80F
http://newartisans.com                          60E1 46C4 BD1A 7AC1 4BA2





  parent reply	other threads:[~2015-12-23 17:54 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-22  1:20 bug#22220: 24.5; no documentation for default process sentinel Ram Bhamidipaty
2015-12-22 17:12 ` Eli Zaretskii
2015-12-22 23:25   ` Ram Bhamidipaty
2015-12-23  3:40     ` Eli Zaretskii
2015-12-23 17:34       ` John Wiegley
2015-12-23 17:45         ` Ram Bhamidipaty
2015-12-23 17:56           ` Eli Zaretskii
2015-12-23 18:05             ` Ram Bhamidipaty
2015-12-23 18:15             ` John Wiegley
2015-12-23 17:54         ` Ram Bhamidipaty [this message]
2015-12-23 18:20           ` 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=BLUPR0101MB1553E8B64A52BA58870DE997BAE60@BLUPR0101MB1553.prod.exchangelabs.com \
    --to=rbhamidipaty@roku.com \
    --cc=22220@debbugs.gnu.org \
    --cc=eliz@gnu.org \
    --cc=jwiegley@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 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.