[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [PATCH 05/10] Remove info arg from several org-babel functions
From: |
Aaron Ecay |
Subject: |
Re: [O] [PATCH 05/10] Remove info arg from several org-babel functions |
Date: |
Thu, 18 Apr 2013 03:07:31 -0400 |
User-agent: |
Notmuch/0.15.2+43~ge848af8 (http://notmuchmail.org) Emacs/24.3.50.1 (x86_64-unknown-linux-gnu) |
Hi Eric,
Thanks for your comments on this and all the patches.
2013ko apirilak 3an, Eric Schulte-ek idatzi zuen:
>
> Aaron Ecay <address@hidden> writes:
>
>> * 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
>>
>> The info arg is threaded through this code, but never used by
>> callers (at least in org code).
>
> The rgrep command disagrees with this last statement.
>
> ./ob-core.el:411: (progn (org-babel-load-in-session current-prefix-arg
> info) 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’d prefer.
--
Aaron Ecay
- Re: [O] [PATCH 03/10] Clean up various org-babel-*-maybe commands, (continued)
[O] [PATCH 05/10] Remove info arg from several org-babel functions, Aaron Ecay, 2013/04/01
[O] [PATCH 08/10] Fix testing/lisp/test-ob-emacs-lisp.el, Aaron Ecay, 2013/04/01
[O] [PATCH 06/10] Use prefix arg in org-edit-special, Aaron Ecay, 2013/04/01
[O] [PATCH 09/10] Remove org-babel-check-confirm-evaluate macro, Aaron Ecay, 2013/04/01
[O] [PATCH 10/10] Document how :var introduces code block dependencies., Aaron Ecay, 2013/04/01