Re: [PATCH] org-babel-insert-result: Fix wrong-type-argument markerp nil

[Bruno Barbier <perso.bruno.barbier@free.fr>]

Ihor Radchenko <yantar92@posteo.net> writes:

> Bruno Barbier <perso.bruno.barbier@free.fr> writes:
>
>> Here is a small patch for a corner case in `org-babel-insert-result'.
>
> Thanks, but could you please explain a bit more about this corner case?

> `end' must be always defined because
> (setq end (copy-marker (point) t))
> must be executed.

If the body of the unwind-protect throw an error, the variable 'end'
might still be nil. It's set to nil when declared with the 'let' form.

>
> Your patch, if addressing a valid scenario, is not treating the root
> cause; just the symptom.

You're right. I was investigating an other bug. That
'wrong-type-argument markerp nil' was just hiding the real bug to me.

Bruno