From mboxrd@z Thu Jan 1 00:00:00 1970 From: Charles Berry Subject: Re: [BUG] org-confirm-babel-evaluate breaks inline src blocks Date: Sat, 23 Nov 2013 18:24:17 +0000 (UTC) Message-ID: References: <877gc0ihd9.fsf@gmail.com> <87y54ff5ma.fsf@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:51027) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VkHsq-0004pg-SD for emacs-orgmode@gnu.org; Sat, 23 Nov 2013 13:24:50 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VkHsl-0006Os-1I for emacs-orgmode@gnu.org; Sat, 23 Nov 2013 13:24:44 -0500 Received: from plane.gmane.org ([80.91.229.3]:46447) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VkHsk-0006Oj-R9 for emacs-orgmode@gnu.org; Sat, 23 Nov 2013 13:24:38 -0500 Received: from list by plane.gmane.org with local (Exim 4.69) (envelope-from ) id 1VkHsj-00048P-EE for emacs-orgmode@gnu.org; Sat, 23 Nov 2013 19:24:37 +0100 Received: from 137.110.38.30 ([137.110.38.30]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Sat, 23 Nov 2013 19:24:37 +0100 Received: from ccberry by 137.110.38.30 with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Sat, 23 Nov 2013 19:24:37 +0100 List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org To: emacs-orgmode@gnu.org Eric Schulte gmail.com> writes: > > Charles Berry ucsd.edu> writes: > > > Eric Schulte gmail.com> writes: > > > >> > >> Charles Berry ucsd.edu> writes: > >> > > [snip] > Indeed the fix was to set this element of info to point to the front of > the inline src block. I imagine that you probably don't have the > patched version of the relevant function loaded. Please try M-x > describe-function on org-babel-get-src-block-info, then jump to the > definition of said function where you should see the following code > which now sets the location of the inline src block head into the last > element of the info list. > > ;; inline source block > (when (org-babel-get-inline-src-block-matches) > (setq head (match-beginning 0)) > (setq info (org-babel-parse-inline-src-block-match)))) > That _is_ what I had loaded, but ... org-babel-get-src-block-info is *not* *called* for inline src blocks. org-babel-exp-non-block-elements doesn't call org-babel-get-src-block-info. Instead, it has this (let* ((info (org-babel-parse-inline-src-block-match)) for inline src blocks. So changing org-babel-get-src-block-infodoes not fix the inline src block issue. === Juat to be sure that I understood what was happening, I instrumented org-babel-get-src-block-info and ran the ECM I posted at the start of this thread. org-babel-get-src-block-info got called twice for the first src block, then I got prompted "Evaluate this emacs-lisp code block on your system? " If I answer 'n' (skip the block redefining org-confirm-babel-evaluate, then I get prompted for the inline src block _without_ org-babel-get-src-block-info being called again. If I answer 'y' (redefine org-confirm-babel-evaluate as a lambda form), I get the error message. org-babel-exp-results: Wrong type argument: integer-or-marker-p, nil HTH, Chuck