GNU bug report logs - #43489
[PATCH] Don't signal scan-error when moving by sexp interactively

Previous Next

Full log

View this message in rfc822 format

From: João Távora <joaotavora <at> gmail.com>
To: Mattias Engdegård <mattiase <at> acm.org>
Cc: 43489-done <at> debbugs.gnu.org, Lars Ingebrigtsen <larsi <at> gnus.org>, Stefan Monnier <monnier <at> iro.umontreal.ca>
Subject: bug#43489: [PATCH] Don't signal scan-error when moving by sexp interactively
Date: Thu, 24 Sep 2020 16:58:36 +0100

[Message part 1 (text/plain, inline)]

On Thu, Sep 24, 2020 at 4:50 PM Mattias Engdegård <mattiase <at> acm.org> wrote:

> Let's hope the person debugging isn't too much bothered by typing M-:
(forward-sexp) instead of C-M-f, don't you think

Sure, just a modest +600% increase in keys typed :-)

> Tthere's nothing unusual about commands that use
> lower-level functions in their implementation and need to
> catch errors raised by those functions.

I disagree Mattias.  I think swallowing errors is bad, generally.
Swallowing errors because you wanted to change the way
they're textually represented is worse. It's fixing the problem
in the wrong place, simple as that. Simply because by
fixing the problem it creates a new one, however small
you deem it. These solutions never satisfy me.

But I personally hope _not_ to be terribly bitten by this. After all,
most of elec-pair.el is already done, and quite stable, so I haven't
been messing with this stuff for a while now.

Which reminds me...  Maybe you could just use
condition-case-unless-debug.  Then it will be
just textually ugly but not functionally so.

João

[Message part 2 (text/html, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.