From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED.blaine.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.devel Subject: Re: Backtrace printing in batch mode ignores all customizations Date: Tue, 14 Jan 2020 21:57:06 +0200 Message-ID: <831rs16cbx.fsf@gnu.org> References: <83h80y59l6.fsf@gnu.org> Injection-Info: blaine.gmane.org; posting-host="blaine.gmane.org:195.159.176.226"; logging-data="84280"; mail-complaints-to="usenet@blaine.gmane.org" Cc: monnier@iro.umontreal.ca, emacs-devel@gnu.org To: Paul Pogonyshev Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Tue Jan 14 20:57:28 2020 Return-path: Envelope-to: ged-emacs-devel@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by blaine.gmane.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1irSJc-000LXc-2K for ged-emacs-devel@m.gmane-mx.org; Tue, 14 Jan 2020 20:57:28 +0100 Original-Received: from localhost ([::1]:45376 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1irSJa-0005i6-DE for ged-emacs-devel@m.gmane-mx.org; Tue, 14 Jan 2020 14:57:26 -0500 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:41298) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1irSJ6-0005Gt-Gg for emacs-devel@gnu.org; Tue, 14 Jan 2020 14:56:57 -0500 Original-Received: from fencepost.gnu.org ([2001:470:142:3::e]:52012) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1irSJ6-0000hl-8Q; Tue, 14 Jan 2020 14:56:56 -0500 Original-Received: from [176.228.60.248] (port=4896 helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1irSJ5-0000pB-2Z; Tue, 14 Jan 2020 14:56:55 -0500 In-reply-to: (message from Paul Pogonyshev on Tue, 14 Jan 2020 18:51:31 +0100) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Original-Sender: "Emacs-devel" Xref: news.gmane.org gmane.emacs.devel:244257 Archived-At: > From: Paul Pogonyshev > Date: Tue, 14 Jan 2020 18:51:31 +0100 > Cc: Stefan Monnier , Emacs developers > > > Paul said earlier that the latter condition, which tests > > terminal-name, is always correct in batch mode. If that is indeed so, > > then why do we need to test noninteractive as well? it's redundant, > > no? > > It is true, but it feels sort of a side-effect to me. You even say > yourself later that you don't like the condition as it goes into > low-level internals. It is extremely non-obvious that it is also true > in non-interactive mode. It is as (un)obvious as the test against the name of the initial terminal, because in batch mode we don't delete that terminal. > I think it is better to explicitly have `(or noninteractive' rather > than rely on it implicitly be a subcase of the second `or' operand > (and forget it was meant to be when the second condition is > refactored). As I've said, I'd be okay with removing the test of the terminal name, and using something much less subtle, but I hesitate to ask to do this as part of the change on the released branch, since this "unclean" test is already there. > > And the request is to add a comment explaining the semantics of > > only-backtrace. Not how it is set -- this is clear from the code, -- > > but what does it mean in terms of the code after that which uses the > > value. > > How about this? > > ;; When operating without any user interaction, we are going to > print current > ;; backtrace and kill Emacs, because there is no way to accept > debugging commands. Fine with me, but maybe for even more clarity, mention what we do in addition in the case where user interaction is possible, so that the last part of that sentence is more self-explanatory. Thanks.