[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [PATCH] expose nrepl's timeout setting in ob-clojure.el
From: |
Nicolas Goaziou |
Subject: |
Re: [O] [PATCH] expose nrepl's timeout setting in ob-clojure.el |
Date: |
Tue, 12 Apr 2016 22:18:27 +0200 |
Hello,
Frederick Giasson <address@hidden> writes:
>>> (setq result
>>> (nrepl-dict-get
>>> - (nrepl-sync-request:eval
>>> - expanded (cider-current-connection) (cider-current-session))
>>> + (let ((nrepl-sync-request-timeout
>>> org-babel-clojure-sync-nrepl-timeout))
>>> + (nrepl-sync-request:eval
>>> + expanded (cider-current-connection) (cider-current-session)))
>> You forgot to
>>
>> (defvar nrepl-sync-request-timeout)
>
> This one is defined in the nREPL package. Maybe there is something
> that I don't understand, but do I have to re-defined it here?
The byte-compiler complains if a variable is let-bound but yet not used
in the body. You don't need to define it again but tell the
byte-compiler it is dynamically scoped (using `defvar' without a value).
Regards,
--
Nicolas Goaziou
[O] [PATCH] new :async feature for org-babel-clojure, Frederick Giasson, 2016/04/06