[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Emacs-diffs] master 425c281 1/3: Allow t as a catch-all condition-c
From: |
Noam Postavsky |
Subject: |
Re: [Emacs-diffs] master 425c281 1/3: Allow t as a catch-all condition-case handler (Bug#24618) |
Date: |
Wed, 5 Sep 2018 10:38:11 -0400 |
On 4 September 2018 at 22:46, Stefan Monnier <address@hidden> wrote:
>> +A CONDITION-NAME of t applies to any error symbol.
>
> All error symbols are caught by `error` already. Those extra conditions
> caught by `t` are non-error conditions
Hmm, right. The rest of the docstring talks only about errors though.
Should we change the whole thing? While technically more accurate,
saying "Regain control when a signal is signaled" sounds a bit funny.
Otherwise, fixing just that sentence:
A handler is applicable to an error if CONDITION-NAME is one of the
-error's condition names. A CONDITION-NAME of t applies to any error
-symbol. If an error happens, the first applicable handler is run.
+error's condition names. A CONDITION-NAME of t applies to any condition
+name. If an error happens, the first applicable handler is run.
> (the only one I know of is `quit`, BTW).
That, and possible silly code which does something like:
(signal 'not-an-error-symbol "oops")
- Re: [Emacs-diffs] master 425c281 1/3: Allow t as a catch-all condition-case handler (Bug#24618), Stefan Monnier, 2018/09/04
- Re: [Emacs-diffs] master 425c281 1/3: Allow t as a catch-all condition-case handler (Bug#24618),
Noam Postavsky <=
- Re: [Emacs-diffs] master 425c281 1/3: Allow t as a catch-all condition-case handler (Bug#24618), Stefan Monnier, 2018/09/05
- Re: [Emacs-diffs] master 425c281 1/3: Allow t as a catch-all condition-case handler (Bug#24618), Noam Postavsky, 2018/09/06
- Re: [Emacs-diffs] master 425c281 1/3: Allow t as a catch-all condition-case handler (Bug#24618), Stefan Monnier, 2018/09/06
- Re: [Emacs-diffs] master 425c281 1/3: Allow t as a catch-all condition-case handler (Bug#24618), Davis Herring, 2018/09/06
- Re: [Emacs-diffs] master 425c281 1/3: Allow t as a catch-all condition-case handler (Bug#24618), Noam Postavsky, 2018/09/07
- Re: [Emacs-diffs] master 425c281 1/3: Allow t as a catch-all condition-case handler (Bug#24618), Noam Postavsky, 2018/09/16
- Re: [Emacs-diffs] master 425c281 1/3: Allow t as a catch-all condition-case handler (Bug#24618), Eli Zaretskii, 2018/09/07
- Re: [Emacs-diffs] master 425c281 1/3: Allow t as a catch-all condition-case handler (Bug#24618), Davis Herring, 2018/09/07
- Re: [Emacs-diffs] master 425c281 1/3: Allow t as a catch-all condition-case handler (Bug#24618), Eli Zaretskii, 2018/09/07
- RE: [Emacs-diffs] master 425c281 1/3: Allow t as a catch-all condition-case handler (Bug#24618), Drew Adams, 2018/09/07