Re: [O] [PATCH 07/10] Simplify org-babel-execute-src-block

emacs-orgmode

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [O] [PATCH 07/10] Simplify org-babel-execute-src-block

From:	Achim Gratz
Subject:	Re: [O] [PATCH 07/10] Simplify org-babel-execute-src-block
Date:	Tue, 02 Apr 2013 21:41:26 +0200
User-agent:	Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux)

Aaron Ecay writes:
> * lisp/ob-core.el (org-babel-execute-src-block): Simplify control flow
>
> Avoid potential duplication of org-babel-process-params call.  Also
> makes the code simpler.

You may be changing semantics here.  I'm not entirely certain if the
current way of dealing with the the unmerged and merged parameters and
the info block is necessary, but I'd be wary of such a change.

Regards,
Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

Wavetables for the Waldorf Blofeld:
http://Synth.Stromeko.net/Downloads.html#BlofeldUserWavetables

[Prev in Thread]

Current Thread

[Next in Thread]

Re: [O] [PATCH 03/10] Clean up various org-babel-*-maybe commands, (continued)
- [O] [PATCH 02/10] Clean up org-babel-expand-body: functions for awk and picolisp, Aaron Ecay, 2013/04/01
  - Re: [O] [PATCH 02/10] Clean up org-babel-expand-body: functions for awk and picolisp, Eric Schulte, 2013/04/03
- [O] [PATCH 07/10] Simplify org-babel-execute-src-block, Aaron Ecay, 2013/04/01
  - Re: [O] [PATCH 07/10] Simplify org-babel-execute-src-block, Achim Gratz <=
    - Re: [O] [PATCH 07/10] Simplify org-babel-execute-src-block, Eric Schulte, 2013/04/03
    - Re: [O] [PATCH 07/10] Simplify org-babel-execute-src-block, Achim Gratz, 2013/04/03
    - Re: [O] [PATCH 07/10] Simplify org-babel-execute-src-block, Eric Schulte, 2013/04/03
- [O] [PATCH 05/10] Remove info arg from several org-babel functions, Aaron Ecay, 2013/04/01
  - Re: [O] [PATCH 05/10] Remove info arg from several org-babel functions, Eric Schulte, 2013/04/03
    - Re: [O] [PATCH 05/10] Remove info arg from several org-babel functions, Aaron Ecay, 2013/04/18
- [O] [PATCH 08/10] Fix testing/lisp/test-ob-emacs-lisp.el, Aaron Ecay, 2013/04/01
  - Re: [O] [PATCH 08/10] Fix testing/lisp/test-ob-emacs-lisp.el, Eric Schulte, 2013/04/03
- [O] [PATCH 06/10] Use prefix arg in org-edit-special, Aaron Ecay, 2013/04/01
  - Re: [O] [PATCH 06/10] Use prefix arg in org-edit-special, Eric Schulte, 2013/04/03

Prev by Date: Re: [O] [PATCH 03/10] Clean up various org-babel-*-maybe commands
Next by Date: Re: [O] [PATCH 09/10] Remove org-babel-check-confirm-evaluate macro
Previous by thread: [O] [PATCH 07/10] Simplify org-babel-execute-src-block
Next by thread: Re: [O] [PATCH 07/10] Simplify org-babel-execute-src-block
Index(es):
- Date
- Thread