all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Philippe Vaucher <philippe.vaucher@gmail.com>
Cc: monnier@iro.umontreal.ca, 24555@debbugs.gnu.org
Subject: bug#24555: [PATCH] Remove unused variable `command-debug-status'
Date: Thu, 29 Sep 2016 18:04:31 +0300	[thread overview]
Message-ID: <83fuoioigw.fsf@gnu.org> (raw)
In-Reply-To: <CAGK7Mr5SrraQbJ+LtgA9M8kyqHarePE4wbmN=L2FdE6CKpEbsQ@mail.gmail.com> (message from Philippe Vaucher on Thu, 29 Sep 2016 08:50:08 +0200)

> From: Philippe Vaucher <philippe.vaucher@gmail.com>
> Date: Thu, 29 Sep 2016 08:50:08 +0200
> Cc: Eli Zaretskii <eliz@gnu.org>, 24555@debbugs.gnu.org
> 
> > Can't think of any situation where I'd want/need that.  The name seems
> > to indicate it could be used that way in the context of debugging, but
> > even in that kind of scenario I'm unable to come up with a use-case.
> 
> That's why I asked what was the use of such variable... even if we
> restore the code that sets it, it is not used anywhere else.
> 
> The commit history suggest it might have been useful near the start of
> the development of Emacs, but I don't think it is anymore.
> 
> I vote for dropping it.

I don't think we can drop features just because no use case presents
itself for the moment.  IME, a debugging aid could be of no use until
you are in a situation where it is something to kill for.

But we don't need to argue about this now, because the way we remove
features from Emacs is by first declaring them obsolete, and keeping
them like that for a few releases; we don't just drop them without a
previous notice.

So I'm okay with declaring this variable obsolete, and even removing
its documentation from the manual, but we do need to restore the
specbind call that was deleted.

Philippe, can you do this, please?  It should be done on the emacs-25
branch, since the specbind was deleted in 25.1.

Thanks.





  reply	other threads:[~2016-09-29 15:04 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-28 12:39 bug#24555: [PATCH] Remove unused variable `command-debug-status' Philippe Vaucher
2016-09-28 14:58 ` Eli Zaretskii
2016-09-28 16:14   ` Stefan Monnier
2016-09-28 17:24     ` Philippe Vaucher
2016-09-28 20:11       ` Eli Zaretskii
2016-09-29  0:16         ` Stefan Monnier
2016-09-29  6:50           ` Philippe Vaucher
2016-09-29 15:04             ` Eli Zaretskii [this message]
2016-09-30  9:08               ` Philippe Vaucher
2016-10-01 13:09                 ` Philippe Vaucher
2016-10-01 15:50                   ` Eli Zaretskii
2016-10-01 16:12                     ` Philippe Vaucher
2016-10-01 17:39                       ` Eli Zaretskii
2016-10-02 17:08                         ` Philippe Vaucher
2016-10-02 18:32                           ` Noam Postavsky
2016-10-02 18:48                             ` Philippe Vaucher
2016-10-02 19:41                         ` Philippe Vaucher
2016-10-03  7:13                           ` Eli Zaretskii
2016-10-03  7:37                             ` Philippe Vaucher
2016-10-03  7:42                               ` Philippe Vaucher
2016-10-04 14:43                                 ` Eli Zaretskii
2016-10-04 15:18                                   ` Philippe Vaucher
2016-10-04 15:46                                     ` Eli Zaretskii
2016-10-04 15:54                                       ` Philippe Vaucher
2016-09-28 20:05     ` Eli Zaretskii
2016-09-29  0:12       ` Stefan Monnier

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=83fuoioigw.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=24555@debbugs.gnu.org \
    --cc=monnier@iro.umontreal.ca \
    --cc=philippe.vaucher@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.