[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [PATCH] ob-eval: display error fix
From: |
Ruijie Yu |
Subject: |
Re: [O] [PATCH] ob-eval: display error fix |
Date: |
Fri, 27 Jan 2023 09:48:31 +0800 |
User-agent: |
mu4e 1.8.13; emacs 29.0.60 |
Ping -- I don't think anyone else has replied / reviewed this patch yet,
unless it has happened in a different thread.
Best,
RY
Andreas Gerler <baron@bundesbrandschatzamt.de> writes:
> [[PGP Signed Part:Undecided]]
>
> [2. text/x-patch; 0001-lisp-ob-eval.el-Display-error-fix.patch]...
>
>
>
> Hi Ruijie,
>
> thanks for asking.
> I combined them now and tested again all variants.
>
> Andreas
>
>> On 7. Jan 2023, at 22:33, Ruijie Yu via General discussions about Org-mode.
>> <emacs-orgmode@gnu.org> wrote:
>>
>>
>> Hi Andreas,
>>
>> - (display-buffer buf))
>> + (if org-babel-eval-error-display-notify
>> + (display-buffer buf)
>> + (if (or (not (numberp exit-code))
>> + (> exit-code 0))
>> + (display-buffer buf))))
>>
>> Quick question, does it help or hurt readability if we change this
>> nested `if' into a `cond' expression?
>>
>> Or, alternatively, since both the outer then branch and the inner then
>> branch are the same expression, maybe we can just combine all three
>> conditions into a single `or' call?
>>
>> Something like
>>
>> (when (or org-babel-eval-error-display-notify
>> (not (numberp exit-code))
>> (> exit-code 0))
>> (display-buffer buf))
>>
>> Best,
>>
>>
>> RY
>>
>
> [[End of PGP Signed Part]]