GNU bug report logs - #22220
24.5; no documentation for default process sentinel

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 22220 in the body.
You can then email your comments to 22220 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Ram Bhamidipaty <rbhamidipaty <at> roku.com>
To: "bug-gnu-emacs <at> gnu.org" <bug-gnu-emacs <at> gnu.org>
Subject: 24.5; no documentation for default process sentinel
Date: Tue, 22 Dec 2015 01:20:28 +0000

[Message part 1 (text/plain, inline)]

Emacs has the concept of a default process sentinel - but this is not documented in the emacs lisp reference manual.

[Message part 2 (text/html, inline)]

Message #8 received at 22220 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ram Bhamidipaty <rbhamidipaty <at> roku.com>
Cc: 22220 <at> debbugs.gnu.org
Subject: Re: bug#22220: 24.5; no documentation for default process sentinel
Date: Tue, 22 Dec 2015 19:12:41 +0200

> From: Ram Bhamidipaty <rbhamidipaty <at> roku.com>
> Date: Tue, 22 Dec 2015 01:20:28 +0000
> 
> Emacs has the concept of a default process sentinel - but this is not
> documented in the emacs lisp reference manual.

The default sentinel is documented with the set-process-sentinel
function:

 -- Function: set-process-sentinel process sentinel
     This function associates SENTINEL with PROCESS.  If SENTINEL is
     `nil', then the process will have the default sentinel, which
     inserts a message in the process's buffer when the process status
     changes.

Maybe this is not prominent enough, and we should say this in a few
more places.  Other than that, what else do you think should be said
about the default sentinel?

Thanks.

Message #11 received at 22220 <at> debbugs.gnu.org (full text, mbox):

From: Ram Bhamidipaty <rbhamidipaty <at> roku.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: "22220 <at> debbugs.gnu.org" <22220 <at> debbugs.gnu.org>
Subject: RE: bug#22220: 24.5; no documentation for default process sentinel
Date: Tue, 22 Dec 2015 23:25:28 +0000

Hi there,

Thank you for responding.

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

-Ram


-----Original Message-----
From: Eli Zaretskii [mailto:eliz <at> gnu.org] 
Sent: Tuesday, December 22, 2015 9:13 AM
To: Ram Bhamidipaty <rbhamidipaty <at> roku.com>
Cc: 22220 <at> debbugs.gnu.org
Subject: Re: bug#22220: 24.5; no documentation for default process sentinel

> From: Ram Bhamidipaty <rbhamidipaty <at> roku.com>
> Date: Tue, 22 Dec 2015 01:20:28 +0000
> 
> Emacs has the concept of a default process sentinel - but this is not
> documented in the emacs lisp reference manual.

The default sentinel is documented with the set-process-sentinel
function:

 -- Function: set-process-sentinel process sentinel
     This function associates SENTINEL with PROCESS.  If SENTINEL is
     `nil', then the process will have the default sentinel, which
     inserts a message in the process's buffer when the process status
     changes.

Maybe this is not prominent enough, and we should say this in a few
more places.  Other than that, what else do you think should be said
about the default sentinel?

Thanks.

Message #14 received at 22220 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ram Bhamidipaty <rbhamidipaty <at> roku.com>
Cc: 22220 <at> debbugs.gnu.org
Subject: Re: bug#22220: 24.5; no documentation for default process sentinel
Date: Wed, 23 Dec 2015 05:40:00 +0200

> From: Ram Bhamidipaty <rbhamidipaty <at> roku.com>
> CC: "22220 <at> debbugs.gnu.org" <22220 <at> debbugs.gnu.org>
> Date: Tue, 22 Dec 2015 23:25:28 +0000
> 
> It would be nice if the default process sentinel were explicitly documented - at least the name and formal description. 

The description is what I cited from the manual; there's nothing else
to be said about that, because that's all that the default sentinel
does.

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?

Message #17 received at 22220 <at> debbugs.gnu.org (full text, mbox):

From: John Wiegley <jwiegley <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 22220 <at> debbugs.gnu.org, Ram Bhamidipaty <rbhamidipaty <at> roku.com>
Subject: Re: bug#22220: 24.5; no documentation for default process sentinel
Date: Wed, 23 Dec 2015 09:34:09 -0800

>>>>> Eli Zaretskii <eliz <at> 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

Message #20 received at 22220 <at> debbugs.gnu.org (full text, mbox):

From: Ram Bhamidipaty <rbhamidipaty <at> roku.com>
To: John Wiegley <jwiegley <at> gmail.com>, Eli Zaretskii <eliz <at> gnu.org>
Cc: "22220 <at> debbugs.gnu.org" <22220 <at> debbugs.gnu.org>
Subject: RE: bug#22220: 24.5; no documentation for default process sentinel
Date: Wed, 23 Dec 2015 17:45:22 +0000

This is what I ran into: I am working on some code using the emacs process facility. And I started seeing messages (from the default sentinel) that I did not expect. When I _skimmed_ through the sentinel section I did not notice that a default sentinel is added to all processes. I acknowledge that skimming the docs was not the right thing to do - since I missed the fact that there is a default sentinel.

From my point of view - I would have been nice - if the default sentinel info were a bit more prominent.

So - I proposed that the name and behavior of the default sentinel be documented. It might not be needed - in the sense that the name itself is not required - but if there were a block of text in the info section like "-- Function: xxx default-sentinel" that would be more prominent - and probably harder to miss.

-Ram

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

>>>>> Eli Zaretskii <eliz <at> 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

Message #23 received at 22220 <at> debbugs.gnu.org (full text, mbox):

From: Ram Bhamidipaty <rbhamidipaty <at> roku.com>
To: John Wiegley <jwiegley <at> gmail.com>, Eli Zaretskii <eliz <at> gnu.org>
Cc: "22220 <at> debbugs.gnu.org" <22220 <at> debbugs.gnu.org>
Subject: RE: bug#22220: 24.5; no documentation for default process sentinel
Date: Wed, 23 Dec 2015 17:54:12 +0000

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 <at> gmail.com] 
Sent: Wednesday, December 23, 2015 9:34 AM
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: Ram Bhamidipaty <rbhamidipaty <at> roku.com>; 22220 <at> debbugs.gnu.org
Subject: Re: bug#22220: 24.5; no documentation for default process sentinel

>>>>> Eli Zaretskii <eliz <at> 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

Message #26 received at 22220 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ram Bhamidipaty <rbhamidipaty <at> roku.com>
Cc: jwiegley <at> gmail.com, 22220 <at> debbugs.gnu.org
Subject: Re: bug#22220: 24.5; no documentation for default process sentinel
Date: Wed, 23 Dec 2015 19:56:59 +0200

> From: Ram Bhamidipaty <rbhamidipaty <at> roku.com>
> CC: "22220 <at> debbugs.gnu.org" <22220 <at> debbugs.gnu.org>
> Date: Wed, 23 Dec 2015 17:45:22 +0000
> 
> This is what I ran into: I am working on some code using the emacs process facility. And I started seeing messages (from the default sentinel) that I did not expect. When I _skimmed_ through the sentinel section I did not notice that a default sentinel is added to all processes. I acknowledge that skimming the docs was not the right thing to do - since I missed the fact that there is a default sentinel.

OK, I already have a patch that will mention the default sentinel in a
few more places, and I think it will fit that bill.  I will commit it
soon.

Thanks.

Message #29 received at 22220 <at> debbugs.gnu.org (full text, mbox):

From: Ram Bhamidipaty <rbhamidipaty <at> roku.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: "jwiegley <at> gmail.com" <jwiegley <at> gmail.com>,
 "22220 <at> debbugs.gnu.org" <22220 <at> debbugs.gnu.org>
Subject: RE: bug#22220: 24.5; no documentation for default process sentinel
Date: Wed, 23 Dec 2015 18:05:15 +0000

:-) Thank you!
-Ram

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

> From: Ram Bhamidipaty <rbhamidipaty <at> roku.com>
> CC: "22220 <at> debbugs.gnu.org" <22220 <at> debbugs.gnu.org>
> Date: Wed, 23 Dec 2015 17:45:22 +0000
> 
> This is what I ran into: I am working on some code using the emacs process facility. And I started seeing messages (from the default sentinel) that I did not expect. When I _skimmed_ through the sentinel section I did not notice that a default sentinel is added to all processes. I acknowledge that skimming the docs was not the right thing to do - since I missed the fact that there is a default sentinel.

OK, I already have a patch that will mention the default sentinel in a
few more places, and I think it will fit that bill.  I will commit it
soon.

Thanks.

Message #32 received at 22220 <at> debbugs.gnu.org (full text, mbox):

From: John Wiegley <jwiegley <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: jwiegley <at> gmail.com, 22220 <at> debbugs.gnu.org,
 Ram Bhamidipaty <rbhamidipaty <at> roku.com>
Subject: Re: bug#22220: 24.5; no documentation for default process sentinel
Date: Wed, 23 Dec 2015 10:15:49 -0800

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

> OK, I already have a patch that will mention the default sentinel in a few
> more places, and I think it will fit that bill. I will commit it soon.

Thank you, Eli!

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

Message #37 received at 22220-done <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ram Bhamidipaty <rbhamidipaty <at> roku.com>
Cc: jwiegley <at> gmail.com, 22220-done <at> debbugs.gnu.org
Subject: Re: bug#22220: 24.5; no documentation for default process sentinel
Date: Wed, 23 Dec 2015 20:20:59 +0200

> From: Ram Bhamidipaty <rbhamidipaty <at> roku.com>
> CC: "22220 <at> debbugs.gnu.org" <22220 <at> debbugs.gnu.org>
> Date: Wed, 23 Dec 2015 17:54:12 +0000
> 
> 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.

It's okay: the change I've just committed adds that as well.

> 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.

It is now.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.