From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.devel Subject: Re: Removing redisplay-dont-pause Date: Sun, 17 Nov 2024 09:11:01 +0200 Message-ID: <86ed3awd16.fsf@gnu.org> References: Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="5305"; mail-complaints-to="usenet@ciao.gmane.io" Cc: emacs-devel@gnu.org, monnier@iro.umontreal.ca To: Gerd =?utf-8?Q?M=C3=B6llmann?= Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Sun Nov 17 08:11:56 2024 Return-path: Envelope-to: ged-emacs-devel@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1tCZRs-0001GZ-Ig for ged-emacs-devel@m.gmane-mx.org; Sun, 17 Nov 2024 08:11:56 +0100 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tCZR4-00077c-0r; Sun, 17 Nov 2024 02:11:06 -0500 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tCZR2-00077F-Ae for emacs-devel@gnu.org; Sun, 17 Nov 2024 02:11:04 -0500 Original-Received: from fencepost.gnu.org ([2001:470:142:3::e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tCZR1-0007ev-T0; Sun, 17 Nov 2024 02:11:03 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org; s=fencepost-gnu-org; h=MIME-version:References:Subject:In-Reply-To:To:From: Date; bh=Uz1XWYy8o/nIHtRBWZTz8jX7QFJiPi5GuZE3oYUPNEY=; b=csPf7DusuU26s2N8BsSB VoTw2rXNkMFqApPx+JxFFbjQAimkly7UYhpbCRJDyRG9vFmLS8vIlUpGhw3U94xt88+M8rDZjhKUl ScT27ClIikZCmYkHC/AVAkKqZsd9a7k4HlaY2OAKiM0YXSFjE8zN+n4O5SdOJyQKgQ9oJN0OT6rdQ AsLpVE2UQj+HmQhJE1ECA02CAAk8yrXi3e1/D9CbYQEZXnm/vKT7I6PVgro7iq6bzMvB1aglZrXUn 84YCUqFxPE89Iqb+yz3seHHdKR+G0MrpIrMkDvL71gtdAqLMYpgJMiJqMsRrrazirlbYfTUlzafE/ jOSCRDr8CqzYvQ==; In-Reply-To: (message from Gerd =?utf-8?Q?M?= =?utf-8?Q?=C3=B6llmann?= on Sun, 17 Nov 2024 06:43:06 +0100) X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.29 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-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.devel:325487 Archived-At: > From: Gerd Möllmann > Cc: Stefan Monnier > Date: Sun, 17 Nov 2024 06:43:06 +0100 > > Eli Zaretskii writes: > > >> branch: scratch/tty-child-frames > >> commit f62d70f52f4f6b7ed158d618bf790df21f171172 > >> Author: Gerd Möllmann > >> Commit: Gerd Möllmann > >> > >> Don't pause display for pending input > >> > >> * src/dispnew.c: Remove display_completed, redisplay_dont_pause, > >> redisplay-dont-pause was declared obsolete in Emacs 24. Remove anything > >> checking pending input, change function signatures accordingly, and so > >> on. > >> > >> * src/keyboard.c (read_char): Don't use redisplay_dont_pause. > >> * src/minibuf.c (read_minibuf): Use new function signatures. > >> * src/xdisp.c: Don't check display_completed. Use new API. > >> > >> * lisp/subr.el (redisplay-dont-pause): Remove declaration. > > > > I don't think this kind of change is appropriate. Feature branches > > should not add/remove features not directly related to the feature > > being developed on the branch. If we want to remove > > redisplay-dont-pause from Emacs (and I'm not yet sure we do), it > > should be discussed on emacs-devel or in a dedicated bug report, not > > silently installed on the branch. > > I guess not many know what this is about, so what is this about? > > One feature of the old redisplay was that it stopped updating the > display when it detected that input was pending, so that it could > process that input ASAP. > > I kept that feature in my redisplay. As a debugging aid, I added > redisplay-dont-pause with which I could turn this feature on and off, > because pausing the display lead to a lot subtle and difficult to debug > bugs. > > ISTR that setting redisplay-dont-pause to t became sort of a secret > semi-popular hack. Hyrum's Law in action, I guess. And 13 years ago or > so, the default of redisplay-dont-pause was changed to t by Eli. > > 10 years ago, Stefan Monnier explained the why > > + /* Contrary to expectations, a value of "false" can be detrimental to > + responsiveness since aborting a redisplay throws away some of the > + work already performed. It's usually more efficient (and gives > + more prompt feedback to the user) to let the redisplay terminate, > + and just completely skip the next command's redisplay (which is > + done regardless of this setting if there's pending input at the > + beginning of the next redisplay). */ > + redisplay_dont_pause = true; > > And finally, Stefan also > > (make-obsolete-variable 'redisplay-dont-pause nil "24.5") > > and removed it from the docs. > > I've removed that pausing feature in my fork and thought I'd bring that > to master via scratch/tty-child-frames, but apparently that's > controversial for some reason I don't understand, and I reverted that. > And so, here we are. Thanks for the summary. So what do people think about removing this variable (and the code supporting it) from Emacs? In particular, does anyone use that variable in a non-default nil value?