On Monday, 12 August 2019 18:47:48 UTC+1, Alex Miller wrote:
>
> You can’t wrap ns and forms into a single do like that. If you google 
> around you can find this issue referred to as the Gilardi scenario.

OK, thanks, Alex.
Is this not covered in https://technomancy.us/143 by the para that starts 
"In Clojure 1.1, the compiler introduced special handling..."?

Anyway I assumed it was OK because the current code already wraps an ns and 
some forms in a do, here: 
https://github.com/bzg/org-mode/blob/d7e12d1df7091563c5f0fe0bd8b2db634d3e87ba/lisp/ob-clojure.el#L118

stardiviner, please could you explain a bit more about your patch? Before 
it (if I understand it) we had the namespace separately and passed it to 
`nrepl-sync-request:eval` 
as the last param. 

Thanks,
Pete


-- 
You received this message because you are subscribed to the Google
Groups "Clojure" group.
To post to this group, send email to clojure@googlegroups.com
Note that posts from new members are moderated - please be patient with your 
first post.
To unsubscribe from this group, send email to
clojure+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/clojure?hl=en
--- 
You received this message because you are subscribed to the Google Groups 
"Clojure" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to clojure+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/clojure/3fabc0cb-69b9-4525-b561-0812870937e7%40googlegroups.com.

Reply via email to