emacs-orgmode
[Top][All Lists]
Advanced

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

Re: [O] [PATCH] add :session support for ob-js.el


From: Nicolas Goaziou
Subject: Re: [O] [PATCH] add :session support for ob-js.el
Date: Sat, 17 Mar 2018 12:14:26 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux)

Hello,

stardiviner <address@hidden> writes:

> updated my code. I followed your suggestions. But modify code one by
> one messed code up. So I merge all related commits into one commit.
> Sorry for this.

Some comments follow.
> +            (result (if (not (string= (cdr (assq :session params)) "none"))

You can integrate the test above in the `cond':

(cond
 ((string= "none" (cdr (assq :session params))) nil)
 ((string= "*JS REPL*" session) ...)
 ...)
> +           (sit-for .5) (goto-char (point-max))

(sit-for .5)
(goto-char (point-max))

> +           (mapc (lambda (var)
> +                         (insert var) (comint-send-input nil t)
> +                         (org-babel-comint-wait-for-output session)
> +                         (sit-for .1) (goto-char (point-max))) var-lines)))
>      session))

  (dolist (var ...)
   ...)

Also, what's your status wrt FSF papers? This cannot fit as
a TINYCHANGE.

Regard,

-- 
Nicolas Goaziou



reply via email to

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