[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#23949: 25.0.95; Regression in handling error caused by (string-match
From: |
Stefan Monnier |
Subject: |
bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil) |
Date: |
Wed, 13 Jul 2016 09:59:33 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1.50 (gnu/linux) |
>> That's a general problem with the use dynamic binding to pass extra
>> parameters: you end up passing them not just to that one function but
>> also to all other functions called from that one.
> So what is the way forward?
I think it's a fairly fundamental problem that's hard/impossible to "fix".
> Fixing just string-match-p and string-match does not seem to the complete
> solution, because I have seen just let-bound dynamic vars at many places.
A way to fix *some* of the problems is to use concurrency (not yet in
"master"), and make the debugger run in another thread so it's not
affected by the dynamic bindings of the code that signaled the error.
For the more general problem, the only "fix" is to try and remove those
uses of dynamic bindings and replace them with something else, which
seems like a very large undertaking and whose benefits are not
necessarily that clear (the problem being to decide what that "something
else" should be, and the fact that this "something else" will come with
its own problems).
In the case of string-match-p I think we should get rid of
inhibit-changing-match-data and implement string-match-p (and
looking-at-p) some other way.
Stefan
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), (continued)
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Eli Zaretskii, 2016/07/12
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Kaushal Modi, 2016/07/12
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Noam Postavsky, 2016/07/12
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Kaushal Modi, 2016/07/12
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Eli Zaretskii, 2016/07/12
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Eli Zaretskii, 2016/07/12
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Eli Zaretskii, 2016/07/12
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Kaushal Modi, 2016/07/12
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Stefan Monnier, 2016/07/12
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Kaushal Modi, 2016/07/13
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil),
Stefan Monnier <=
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Eli Zaretskii, 2016/07/13
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Eli Zaretskii, 2016/07/13
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Eli Zaretskii, 2016/07/13
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Stefan Monnier, 2016/07/13
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Eli Zaretskii, 2016/07/13
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Stefan Monnier, 2016/07/13
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Eli Zaretskii, 2016/07/13
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Stefan Monnier, 2016/07/13
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Andreas Schwab, 2016/07/13
- bug#23949: 25.0.95; Regression in handling error caused by (string-match-p "." nil), Eli Zaretskii, 2016/07/13