Subject: bug#21723: closed (Re: bug#21723: 24.5; Bug in `server-kill-emacs-query-function')

From: [email-masked] (GNU bug Tracking System)
Date: Wed, 21 Oct 2015 16:53:02 +0000

Your bug report

#21723: 24.5; Bug in `server-kill-emacs-query-function'

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
21723: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=21723
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#21723: 24.5; Bug in `server-kill-emacs-query-function'
From: Eli Zaretskii <[email-masked]>
To: Eli Barzilay <[email-masked]>
Cc: [email-masked]
Date: Wed, 21 Oct 2015 19:51:59 +0300
List-Id: <debbugs-submit.debbugs.gnu.org>

> From: Eli Barzilay <[email-masked]>
> Date: Wed, 21 Oct 2015 04:52:01 -0400
> 
> The code in `server-kill-emacs-query-function' (in server.el) has this
> in an `or':
> 
>     (let (live-client)
>       (dolist (proc server-clients)
>         (when (memq t (mapcar 'buffer-live-p (process-get
>                                               proc 'buffers)))
>           (setq live-client t)))
>       live-client)
> 
> IIUC, this looks for a live client, and if there is one, the t result
> means that the yes-or-no-p question in the end will *not* get asked.
> I verified this by:
> 
>   * Starting emacs -Q and in it start the server
> 
>   * Run emacsclient on a random file
> 
>   * Now, C-x C-c exits without asking any questions.
> 
> It seems to me like that condition is reversed, an the last line
> should instead be: (not live-client)

Ouch!  This bug was there for almost 10 years, since the simple loop
was rewritten as dolist, and the negation of the result was dropped in
the process.

Should be fixed now, thanks.

Subject: 24.5; Bug in `server-kill-emacs-query-function'
From: Eli Barzilay <[email-masked]>
To: [email-masked]
Date: Wed, 21 Oct 2015 04:52:01 -0400
List-Id: <debbugs-submit.debbugs.gnu.org>

The code in `server-kill-emacs-query-function' (in server.el) has this
in an `or':

    (let (live-client)
      (dolist (proc server-clients)
        (when (memq t (mapcar 'buffer-live-p (process-get
                                              proc 'buffers)))
          (setq live-client t)))
      live-client)

IIUC, this looks for a live client, and if there is one, the t result
means that the yes-or-no-p question in the end will *not* get asked.
I verified this by:

  * Starting emacs -Q and in it start the server

  * Run emacsclient on a random file

  * Now, C-x C-c exits without asking any questions.

It seems to me like that condition is reversed, an the last line
should instead be: (not live-client)

This will also make the first condition in the function unnecessary,
since if there are no clients, then the loop will not find a live one
and the question will be skipped.

-- 
                    ((x=>x(x))(x=>x(x)))                   Eli Barzilay:
                    http://barzilay.org/                   Maze is Life!