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

2022-11-08 Thread Bruno Barbier
Ihor Radchenko writes: > https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=620a96dd682191f49cbe83dee516bb566b9d0127 Thanks Ihor.

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

2022-11-08 Thread Ihor Radchenko
Ihor Radchenko writes: > https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=7f72807aead9c350d85513702deb04722b65a25b Oops. The right link is https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=620a96dd682191f49cbe83dee516bb566b9d0127 -- Ihor Radchenko // yantar92, Org

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

2022-11-08 Thread Ihor Radchenko
Bruno Barbier writes: >> 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.

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

2022-11-08 Thread Bruno Barbier
Ihor Radchenko writes: > Bruno Barbier 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

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

2022-11-08 Thread Ihor Radchenko
Bruno Barbier 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? > lisp/ob-core.el (org-babel-insert-result): Don't reset the marker > `end' if it is not defined. `end' must be always defined

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

2022-11-08 Thread Bruno Barbier
Hi, Here is a small patch for a corner case in `org-babel-insert-result'. Bruno >From 288e2631ef60eeaee32f1107754aeb521247d54b Mon Sep 17 00:00:00 2001 From: Bruno BARBIER Date: Sun, 6 Nov 2022 13:57:40 +0100 Subject: [PATCH] org-babel-insert-result: Fix wrong-type-argument markerp nil l