From mboxrd@z Thu Jan 1 00:00:00 1970 From: Aaron Ecay Subject: Re: [PATCH 05/10] Remove info arg from several org-babel functions Date: Thu, 18 Apr 2013 03:07:31 -0400 Message-ID: <87ip3ks4ek.fsf@gmail.com> References: <1364794944-13826-1-git-send-email-aaronecay@gmail.com> <1364794944-13826-6-git-send-email-aaronecay@gmail.com> <87k3ojy79r.fsf@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([208.118.235.92]:51172) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1USiwR-0003qA-IR for emacs-orgmode@gnu.org; Thu, 18 Apr 2013 03:07:58 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1USiwP-00020Q-Fg for emacs-orgmode@gnu.org; Thu, 18 Apr 2013 03:07:35 -0400 Received: from mail-vc0-f180.google.com ([209.85.220.180]:40453) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1USiwP-0001zf-9V for emacs-orgmode@gnu.org; Thu, 18 Apr 2013 03:07:33 -0400 Received: by mail-vc0-f180.google.com with SMTP id m16so2129931vca.11 for ; Thu, 18 Apr 2013 00:07:33 -0700 (PDT) In-Reply-To: <87k3ojy79r.fsf@gmail.com> 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: Eric Schulte Cc: emacs-orgmode@gnu.org Hi Eric, Thanks for your comments on this and all the patches. 2013ko apirilak 3an, Eric Schulte-ek idatzi zuen: >=20 > Aaron Ecay writes: >=20 >> * lisp/ob-core.el (org-babel-load-in-session), >> (org-babel-initiate-session), >> (org-babel-switch-to-session) >> (org-babel-switch-to-session-with-code): Remove info optional arg >>=20 >> The info arg is threaded through this code, but never used by >> callers (at least in org code). >=20 > The rgrep command disagrees with this last statement. >=20 > ./ob-core.el:411: (progn (org-babel-load-in-session current-prefix-arg in= fo) t) This was removed in patch #3 of the series. I tried to make all the patches independent of each other, but this dependency did slip in. I can squash the two patches if you=E2=80=99d prefer. --=20 Aaron Ecay