[ClojureScript] Re: cljsbuild/closure compiler options

2014-12-24 Thread tcr1272
Did you run `lein clean lein cljsbuild clean` after the upgrade? Yes. The arithmetic error occurs whenever I have an input arg that is bound. I believe the compiler sees this as a var and not a number and therefore yaks a warning. That said I'm not sure why my last version from 3 months

[ClojureScript] Updating sorted set inside go block

2014-12-24 Thread Sven Richter
Hi, Using the latest core.async (v0.1.346.0-17112a-alpha) updating a sorted set results in an error. I have this code: (defn get-transformations [] (go (let [[ok resp] (! (h/get-async /csv/all-transformations))] ;(when ok (reset! sess/transformations-cur (:transformations resp))

[ClojureScript] Re: [ANN] cuerdas 0.1.0: A string manipulation library for clojure and clojurescript.

2014-12-24 Thread Sean Johnson
Very nice work. I'll be using this! Also, I agree with Noam that there's a fairly common expectation of blank? that started in the Ruby world. Here's my suggestion for what it's worth: blank? - true iff or nil empty? - true iff , false if nil whitespace? true iff or only white space, false

Re: [ClojureScript] Re: [ANN] cuerdas 0.1.0: A string manipulation library for clojure and clojurescript.

2014-12-24 Thread Andrey Antukh
Hi! Thank you very much! You are right about nil handling, it should be documented and proper handled. But I'm not completely convinced with ruby behavior for that, in many other implementations in other languages it is slightly different. In my opinion that is more proper behavior: blank?

[ClojureScript] Re: cljsbuild/closure compiler options

2014-12-24 Thread Thomas Heller
Hey, I just looked at the warnings code and it seems like a mistake that disabling one warning actually disables 3 by accident. Anyways, the warning definitions [1] are all pretty self explanatory, more so if you look at the actual text the produce in the error functions below [1]. You could

[ClojureScript] Re: Updating sorted set inside go block

2014-12-24 Thread Thomas Heller
Cannot say without the rest of the code but I what is in (:transformations resp)? sorted-set doesn't work if one item doesn't compare to another (eg. numbers vs maps). Suppose: (def a (atom #{})) = (var user/a) (reset! a (into (sorted-set) [1 2 2 2 3])) = #{1 2 3} (conj @a {:name test})

[ClojureScript] Re: Om's implementation of om.core/state

2014-12-24 Thread Tobias Kortkamp
On 12/23/2014 08:32 PM, Bob Erb wrote: Checking Om source to see what om.core/state does, I see: (defn state [cursor] (-state cursor)) What's `-state`? I can't see where it's defined anywhere. What am I missing? It's a protocol method from ICursor. See

[ClojureScript] Re: Updating sorted set inside go block

2014-12-24 Thread Sven Richter
Hi Thomas, the code I pasted was maybe a bit misleading. Function one: (defn save-transformation [_] (go (let [trans-name (hel/get-value transformation-name) [ok _] (! (hel/post-async-ch /cvs/save-transformation {:name trans-name

[ClojureScript] Warning: accessing name exports in externs has no effect

2014-12-24 Thread Juan Manuel Gimeno Illa
In an application build from the chestnut template I'm getting this error deploying to heroku: remote:WARNING: /tmp/build_cd01f924207bc6326f564b278526aad7/resources/public/highlightjs/highlight.js:1: WARNING - accessing name exports in externs has no effect. Perhaps you forgot to add

[ClojureScript] Re: ANN: ClojureScript 0.0-2496, cljs.test - a clojure.test port

2014-12-24 Thread Russell Mull
Things that aren't in cljs.test: - with-test - run-tests can take a custom environment parameter. Things that required rebinding a var in clj.test are configured with an entry in the environment. - :reporter, instead of rebinding the report function - :testing-contexts

Re: [ClojureScript] Re: ANN: ClojureScript 0.0-2496, cljs.test - a clojure.test port

2014-12-24 Thread Yehonathan Sharvit
What about the 'are' macro? On Wed, Dec 24, 2014 at 7:38 PM, Russell Mull russell.m...@gmail.com wrote: Things that aren't in cljs.test: - with-test - run-tests can take a custom environment parameter. Things that required rebinding a var in clj.test are configured with an entry in

Re: [ClojureScript] Re: ANN: ClojureScript 0.0-2496, cljs.test - a clojure.test port

2014-12-24 Thread Russell Mull
Yes, it's there. The things I listed are the only differences I could find. The separation of macros makes it a little confusing, but it's pretty easy to find in the source: - https://github.com/clojure/clojurescript/blob/master/src/cljs/cljs/test.cljs -

[ClojureScript] Re: cljsbuild/closure compiler options

2014-12-24 Thread tcr1272
I see, and thanks for the solution with pointers Thomas. Cheers, Tim -- Note that posts from new members are moderated - please be patient with your first post. --- You received this message because you are subscribed to the Google Groups ClojureScript group. To unsubscribe from this group

Re: [ClojureScript] Re: cljsbuild/closure compiler options

2014-12-24 Thread David Nolen
Thanks for pointing this out. Fixed in master thanks to your patch. Also landed a patch which should cover more problematic warning cases around arithmetic. David On Wed, Dec 24, 2014 at 8:04 AM, Thomas Heller th.hel...@gmail.com wrote: Hey, I just looked at the warnings code and it seems

Re: [ClojureScript] Re: ANN: ClojureScript 0.0-2496, cljs.test - a clojure.test port

2014-12-24 Thread Yehonathan Sharvit
Great news David! What is the usual delay (in days) between commit to the master and release? On Wednesday, 24 December 2014 20:18:39 UTC+2, David Nolen wrote: The `are` macro isn't in a current release, however it's in master and will appear in the next one. Otherwise Russell's assessment is

Re: [ClojureScript] Re: ANN: ClojureScript 0.0-2496, cljs.test - a clojure.test port

2014-12-24 Thread David Nolen
Time between releases are pretty variable though they tend to be pretty frequent around new features like cljs.test as issues need to be ironed out. I'll probably cut another release on Friday. David On Wed, Dec 24, 2014 at 1:32 PM, Yehonathan Sharvit vie...@gmail.com wrote: Great news David!

[ClojureScript] Re: Updating sorted set inside go block

2014-12-24 Thread Thomas Heller
That code doesn't help much either since there is still no way to tell what sess/transformations-cur is. I'd suggest printing the value before trying to swap! it, I see no reason anything in there would confuse core.async. (defn save-transformation [_] (go (let [trans-name (hel/get-value

[ClojureScript] Re: Updating sorted set inside go block

2014-12-24 Thread Sven Richter
Hi Thomas, Thanks for taking the time to answer me. Ok, this is the session namespace, reduced to the relevant parts: (ns de.sveri.structconverter.session (:require [reagent.cursor :refer [cur]] [reagent.core :refer [atom]])) (def state (atom {:cur-csv-page nil :files []