From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Lars Ingebrigtsen Newsgroups: gmane.emacs.bugs Subject: bug#43489: [PATCH] Don't signal scan-error when moving by sexp interactively Date: Fri, 18 Sep 2020 15:13:21 +0200 Message-ID: <87k0wri5xq.fsf@gnus.org> References: <64C7CB3A-36F2-4988-9FF9-B99115C9ED13@acm.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="4226"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux) Cc: 43489@debbugs.gnu.org To: Mattias =?UTF-8?Q?Engdeg=C3=A5rd?= Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Fri Sep 18 15:14:25 2020 Return-path: Envelope-to: geb-bug-gnu-emacs@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 1kJGDZ-00010E-2h for geb-bug-gnu-emacs@m.gmane-mx.org; Fri, 18 Sep 2020 15:14:25 +0200 Original-Received: from localhost ([::1]:51632 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kJGDY-0002nW-4c for geb-bug-gnu-emacs@m.gmane-mx.org; Fri, 18 Sep 2020 09:14:24 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:42420) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kJGDC-0002kT-4N for bug-gnu-emacs@gnu.org; Fri, 18 Sep 2020 09:14:02 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]:57928) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1kJGDB-0000VL-QI for bug-gnu-emacs@gnu.org; Fri, 18 Sep 2020 09:14:01 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1kJGDB-0000N0-LV for bug-gnu-emacs@gnu.org; Fri, 18 Sep 2020 09:14:01 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Lars Ingebrigtsen Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Fri, 18 Sep 2020 13:14:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 43489 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 43489-submit@debbugs.gnu.org id=B43489.16004348141376 (code B ref 43489); Fri, 18 Sep 2020 13:14:01 +0000 Original-Received: (at 43489) by debbugs.gnu.org; 18 Sep 2020 13:13:34 +0000 Original-Received: from localhost ([127.0.0.1]:41241 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1kJGCk-0000M8-AS for submit@debbugs.gnu.org; Fri, 18 Sep 2020 09:13:34 -0400 Original-Received: from quimby.gnus.org ([95.216.78.240]:53794) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1kJGCi-0000Lu-KT for 43489@debbugs.gnu.org; Fri, 18 Sep 2020 09:13:33 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnus.org; s=20200322; h=Content-Transfer-Encoding:Content-Type:MIME-Version:Message-ID :In-Reply-To:Date:References:Subject:Cc:To:From:Sender:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=bWx3Dd/pILYsdhwBvOqAA5qkhxveqw171oHkDmAsYL4=; b=hSLlV8DlIxvDFxNFNwrCXc7H1Q LkClkf8xt/Ann37ZO9KH9h3Xm3FxzFBdVPOx/P24wD2lVyd08j43IubGOTw3s8VVbQLgW9CxslA1Y Ed69t/8GnrN5FnnCT8EG+AHN2WCPzXMEMwTWt9k6CcjALwkClsLWA66RGATymC9npqqE=; Original-Received: from cm-84.212.202.86.getinternet.no ([84.212.202.86] helo=xo) by quimby with esmtpsa (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1kJGCZ-00066v-OJ; Fri, 18 Sep 2020 15:13:26 +0200 X-Now-Playing: Stina Nordenstam's _This Is Stina Nordenstam_: "Stations" In-Reply-To: <64C7CB3A-36F2-4988-9FF9-B99115C9ED13@acm.org> ("Mattias =?UTF-8?Q?Engdeg=C3=A5rd?="'s message of "Fri, 18 Sep 2020 13:31:16 +0200") X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.io gmane.emacs.bugs:188297 Archived-At: Mattias Engdeg=C3=A5rd writes: > When moving by sexp (C-M-f, C-M-u and so on) and point is already at a > boundary preventing further movement, Emacs currently signals an > internal error such as > > Scan error: "Containing expression ends prematurely", 5010, 5010 > > or > > Scan error: "Unbalanced parentheses", 5010, 1 > > which is unhelpful and rather looks as if something went wrong in the > internal machinery. Yes, those error messages are confusing in interactive usage. > The attached patch does away with this error when the commands are > invoked interactively; programmatic use of the functions will get the > scan-error just like before. There didn't seem to be much point in > replacing the errors with new messages so the current version of the > patch doesn't. [...] > + (if noerror > + (condition-case _ > + (forward-list arg nil) > + (scan-error (ding))) So you basically just `ding' in interactive usage? I wonder whether this would have any negative effect when people are using these commands in keyboard macros. For instance, if you've recorded a macro that does `M-C-f M-DEL' or something, previously it would signal an error and then stop, while now it'll just continue and delete the wrong thing? --=20 (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no