From mboxrd@z Thu Jan  1 00:00:00 1970
From: Eric Schulte <schulte.eric@gmail.com>
Subject: Re: [PATCH 06/10] Use prefix arg in org-edit-special
Date: Wed, 03 Apr 2013 07:42:06 -0600
Message-ID: <87d2ubzlup.fsf@gmail.com>
References: <1364794944-13826-1-git-send-email-aaronecay@gmail.com>
	<1364794944-13826-7-git-send-email-aaronecay@gmail.com>
Mime-Version: 1.0
Content-Type: text/plain
Return-path: <emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org>
Received: from eggs.gnu.org ([208.118.235.92]:37416)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <schulte.eric@gmail.com>) id 1UNPQ1-0001UW-Jp
	for emacs-orgmode@gnu.org; Wed, 03 Apr 2013 11:16:14 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <schulte.eric@gmail.com>) id 1UNPQ0-0007Tr-67
	for emacs-orgmode@gnu.org; Wed, 03 Apr 2013 11:16:09 -0400
Received: from mail-pb0-f41.google.com ([209.85.160.41]:36548)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <schulte.eric@gmail.com>) id 1UNPPz-0007TX-Sl
	for emacs-orgmode@gnu.org; Wed, 03 Apr 2013 11:16:08 -0400
Received: by mail-pb0-f41.google.com with SMTP id mc17so911099pbc.14
	for <emacs-orgmode@gnu.org>; Wed, 03 Apr 2013 08:16:07 -0700 (PDT)
List-Id: "General discussions about Org-mode." <emacs-orgmode.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/emacs-orgmode>,
	<mailto:emacs-orgmode-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/emacs-orgmode>
List-Post: <mailto:emacs-orgmode@gnu.org>
List-Help: <mailto:emacs-orgmode-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/emacs-orgmode>,
	<mailto:emacs-orgmode-request@gnu.org?subject=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: Aaron Ecay <aaronecay@gmail.com>
Cc: emacs-orgmode@gnu.org

Aaron Ecay <aaronecay@gmail.com> writes:

> * lisp/org.el (org-edit-special): Use prefix arg, as docstring says we
>   do
>

This is beyond my ken.  I'll leave review of this patch to Bastien.

>
> Only makes a difference for src-block editing.
> ---
>  lisp/org.el | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/lisp/org.el b/lisp/org.el
> index 04ce386..1edfbc4 100644
> --- a/lisp/org.el
> +++ b/lisp/org.el
> @@ -19943,7 +19943,7 @@ When in a fixed-width region, call `org-edit-fixed-width-region'.
>  When at an #+INCLUDE keyword, visit the included file.
>  On a link, call `ffap' to visit the link at point.
>  Otherwise, return a user error."
> -  (interactive)
> +  (interactive "P")
>    (let ((element (org-element-at-point)))
>      (assert (not buffer-read-only) nil
>  	    "Buffer is read-only: %s" (buffer-name))
> @@ -19958,8 +19958,9 @@ Otherwise, return a user error."
>               ;; At a src-block with a session and function called with
>               ;; an ARG: switch to the buffer related to the inferior
>               ;; process.
> -             (funcall (intern (concat "org-babel-prep-session:" lang))
> -                      session params)))))
> +             (switch-to-buffer
> +	      (funcall (intern (concat "org-babel-prep-session:" lang))
> +		       session params))))))
>        (keyword
>         (if (member (org-element-property :key element) '("INCLUDE" "SETUPFILE"))
>             (find-file

-- 
Eric Schulte
http://cs.unm.edu/~eschulte