From: Achim Gratz <Stromeko@nexgo.de>
To: emacs-orgmode@gnu.org
Cc: Eric Schulte <schulte.eric@gmail.com>
Subject: Re: Org release 8.2.5g (minor release from maint)
Date: Sat, 25 Jan 2014 08:58:56 +0100 [thread overview]
Message-ID: <87fvocbigv.fsf@Rainer.invalid> (raw)
In-Reply-To: 87wqhtywkx.fsf@Rainer.invalid
[-- Attachment #1: Type: text/plain, Size: 401 bytes --]
Achim Gratz writes:
[…]
> in the second or the cdr of the second element of pre-info might
> directly get the new value spliced in depending on whether the original
> value is used someplace else.
Splicing seems slightly more elegant than list construction, but
pre-info needs to be preserved. Eric, please review the attached patch,
I'm not certain about the current test coverage in that area.
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-ob-lob-do-not-use-cl-at-runtime.patch --]
[-- Type: text/x-patch, Size: 1550 bytes --]
From 024e05c4de3c7598448ee97b0f986562007d5186 Mon Sep 17 00:00:00 2001
From: Achim Gratz <Stromeko@Stromeko.DE>
Date: Sat, 25 Jan 2014 08:53:33 +0100
Subject: [PATCH] ob-lob: do not use cl at runtime
* lisp/ob-lob.el (org-babel-lob-execute): Do not use defun subseq from
cl at runtime. Replace concatenation of sub-sequences by splicing
the modified params list into a copy of info (pre-must info be
preserved).
---
lisp/ob-lob.el | 15 ++++++++-------
1 file changed, 8 insertions(+), 7 deletions(-)
diff --git a/lisp/ob-lob.el b/lisp/ob-lob.el
index c93198a..6480468 100644
--- a/lisp/ob-lob.el
+++ b/lisp/ob-lob.el
@@ -147,13 +147,14 @@ (defun org-babel-lob-execute (info)
;; Do *not* pre-process params for call line
;; hash evaluation, since for a call line :var
;; extension *is* execution.
- (let ((params (nth 2 pre-info)))
- (append (subseq pre-info 0 2)
- (list
- (cons
- (cons :c-var (cdr (assoc :var params)))
- (assq-delete-all :var (copy-tree params))))
- (subseq pre-info 3))))))
+ (let* ((params (nth 2 pre-info))
+ (sha1-nth2 (list
+ (cons
+ (cons :c-var (cdr (assoc :var params)))
+ (assq-delete-all :var (copy-tree params)))))
+ (sha1-info (copy-tree pre-info)))
+ (prog1 sha1-info
+ (setcar (cddr sha1-info) sha1-nth2))))))
(old-hash (when cache-p (org-babel-current-result-hash pre-info)))
(org-babel-current-src-block-location (point-marker)))
(if (and cache-p (equal new-hash old-hash))
--
1.8.5.2
[-- Attachment #3: Type: text/plain, Size: 197 bytes --]
Regards,
Achim.
--
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+
SD adaptations for KORG EX-800 and Poly-800MkII V0.9:
http://Synth.Stromeko.net/Downloads.html#KorgSDada
next prev parent reply other threads:[~2014-01-25 7:59 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-21 13:50 Org release 8.2.5g (minor release from maint) Bastien
2014-01-21 19:16 ` Achim Gratz
2014-01-21 19:25 ` Bastien
2014-01-21 19:30 ` Bastien
2014-01-21 19:35 ` Achim Gratz
2014-01-21 23:39 ` Bastien
2014-01-22 7:52 ` Achim Gratz
2014-01-25 7:58 ` Achim Gratz [this message]
2014-02-26 19:25 ` Achim Gratz
2014-02-26 23:23 ` Eric Schulte
2014-03-01 8:06 ` Bastien
2014-03-01 9:10 ` Achim Gratz
2014-03-01 12:53 ` Nick Dokos
2014-03-03 3:03 ` Eric Schulte
2014-03-05 1:56 ` Eric Abrahamsen
2014-03-05 3:20 ` Nick Dokos
2014-03-06 17:00 ` Eric Schulte
2014-03-07 7:26 ` Bastien
2014-03-07 16:05 ` Richard Lawrence
2014-03-07 21:09 ` Eric Schulte
2014-03-08 4:05 ` Richard Lawrence
2014-03-13 14:55 ` Bastien
2014-03-13 16:28 ` Richard Lawrence
2014-03-21 8:08 ` Bastien
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.orgmode.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87fvocbigv.fsf@Rainer.invalid \
--to=stromeko@nexgo.de \
--cc=emacs-orgmode@gnu.org \
--cc=schulte.eric@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).