bug#59668: 29.0.50; [PATCH] Make 'server-stop-automatically' into a defcustom

[Jim Porter]

On 12/2/2022 6:42 AM, Eli Zaretskii wrote:
> > Date: Thu, 1 Dec 2022 19:30:03 -0800
> > Cc: 59668@debbugs.gnu.org
> > From: Jim Porter <jporterbugs@gmail.com>
> >
> > @@ -1805,7 +1814,7 @@ server-save-buffers-kill-terminal
> >             (t (error "Invalid client frame"))))))
> >   
> >   (defun server-stop-automatically--handle-delete-frame (frame)
> > -  "Handle deletion of FRAME when `server-stop-automatically' is used."
> > +  "Handle deletion of FRAME when `server-stop-automatically' is 
> > `delete-frame'."
> >     (when server-stop-automatically
> >       (if (if (and (processp (frame-parameter frame 'client))
> >                  (eq this-command 'save-buffers-kill-terminal))
> > @@ -1828,7 +1837,7 @@ server-stop-automatically--handle-delete-frame
> >           (delete-frame frame)))))
>
> AFAIU, this delete-frame is called after save-buffers-kill-emacs, which is
> strange: there will be no Emacs to perform this call after that.  What am I
> missing?

Thanks. I think that's just an oversight. I removed it in my patch for 
bug#51993, since this seems more closely-related to that bug.

> > +          (const :tag "When empty" empty)
>
> "When empty" doesn't explain itself well enough.  Can we come up with a
> better description?

How about "When empty (no clients, unsaved files, or processes)"? That 
seems a bit clumsy to me, but I wanted to keep the word "empty" in there 
somehow, since that's the name of the option value.

Attached is an updated patch (rebased on top of my previous patches for 
bug#51993).

0001-Make-server-stop-automatically-into-a-defcustom.patch
Description: Text document