emacs-elpa-diffs
[Top][All Lists]
Advanced

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

[elpa] externals-release/org 1d5e41d 1/9: lisp/ob-core.el (org-babel-eva


From: ELPA Syncer
Subject: [elpa] externals-release/org 1d5e41d 1/9: lisp/ob-core.el (org-babel-eval-headers): Fix elements ordering
Date: Wed, 29 Sep 2021 08:58:00 -0400 (EDT)

branch: externals-release/org
commit 1d5e41d41ba878424eb87115f9ddab3c9edf44a0
Author: Bastien <bzg@gnu.org>
Commit: Bastien <bzg@gnu.org>

    lisp/ob-core.el (org-babel-eval-headers): Fix elements ordering
    
    * lisp/ob-core.el (org-babel-eval-headers): Fix elements ordering.
    (org-babel-merge-params): Tiny refactoring.
    
    This fixes the test `test-ob-lob/call-with-header-arguments' that
    fails after 9f1507bdd.
---
 lisp/ob-core.el | 25 +++++++++----------------
 1 file changed, 9 insertions(+), 16 deletions(-)

diff --git a/lisp/ob-core.el b/lisp/ob-core.el
index ec1191e..4b7de5f 100644
--- a/lisp/ob-core.el
+++ b/lisp/ob-core.el
@@ -609,8 +609,6 @@ multiple blocks are being executed (e.g., in chained 
execution
 through use of the :var header argument) this marker points to
 the outer-most code block.")
 
-(defvar *this*)
-
 (defun org-babel-eval-headers (headers)
   "Compute header list set with HEADERS.
 
@@ -618,11 +616,10 @@ Evaluate all header arguments set to functions prior to 
returning
 the list of header arguments."
   (let ((lst nil))
     (dolist (elem headers)
-      (if (and (cdr elem)
-              (functionp (cdr elem)))
+      (if (and (cdr elem) (functionp (cdr elem)))
           (push `(,(car elem) . ,(funcall (cdr elem))) lst)
         (push elem lst)))
-    lst))
+    (reverse lst)))
 
 (defun org-babel-get-src-block-info (&optional light datum)
   "Extract information from a source block or inline source block.
@@ -2739,21 +2736,17 @@ parameters when merging lists."
                                  results-exclusive-groups
                                  results
                                  (split-string
-                                  (if (stringp value)
-                                      value
-                                    (if (functionp value)
-                                        (funcall value)
-                                      (eval value t)))))))
+                                  (cond ((stringp value) value)
+                                         ((functionp value) (funcall value))
+                                         (t (eval value t)))))))
          (`(:exports . ,value)
           (setq exports (funcall merge
                                  exports-exclusive-groups
                                  exports
-                                 (split-string (or
-                                                (if value
-                                                    (if (functionp value)
-                                                        (funcall value)
-                                                      value)
-                                                  ""))))))
+                                 (split-string
+                                   (cond ((and value (functionp value)) 
(funcall value))
+                                         (value value)
+                                         (t ""))))))
          ;; Regular keywords: any value overwrites the previous one.
          (_ (setq params (cons pair (assq-delete-all (car pair) params)))))))
     ;; Handle `:var' and clear out colnames and rownames for replaced



reply via email to

[Prev in Thread] Current Thread [Next in Thread]