Skip to content

Commit

Permalink
rename run! macro to react!
Browse files Browse the repository at this point in the history
under [org.clojure/clojurescript "0.0-3308"]
it was shadowing clojure.core/run! and causing warning:
WARNING: run! already refers to: #'clojure.core/run! in namespace: reagent.ratom, being replaced by: #'reagent.ratom/run!
  • Loading branch information
darwin committed Aug 18, 2015
1 parent 3a93bd6 commit 2ed9291
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 19 deletions.
2 changes: 1 addition & 1 deletion src/reagent/ratom.clj
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
`(reagent.ratom/make-reaction
(fn [] ~@body)))

(defmacro run!
(defmacro react!
"Runs body immediately, and runs again whenever atoms deferenced in the body change. Body should side effect."
[& body]
`(let [co# (reagent.ratom/make-reaction (fn [] ~@body)
Expand Down
16 changes: 8 additions & 8 deletions test/reagenttest/testcursor.cljs
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
(ns reagenttest.testcursor
(:require [cljs.test :as t :refer-macros [is deftest testing]]
[reagent.ratom :as rv :refer-macros [run! reaction]]
[reagent.ratom :as rv :refer-macros [react! reaction]]
[reagent.debug :refer-macros [dbg]]
[reagent.core :as r :refer [atom]]))

Expand All @@ -23,7 +23,7 @@
res (reaction
(swap! count inc)
@sv @c2 @comp)
const (run!
const (react!
(reset! out @res))]
(is (= @count 1) "constrain ran")
(is (= @out 2))
Expand Down Expand Up @@ -62,7 +62,7 @@
(let [!ctr-base (rv/atom {:x {:y 0 :z 0}})
!counter (r/cursor !ctr-base [:x :y])
!signal (rv/atom "All I do is change")
co (run!
co (react!
;;when I change...
@!signal
;;update the counter
Expand Down Expand Up @@ -99,7 +99,7 @@
c (reaction
(swap! c-changed inc)
(+ 10 @a2))
res (run!
res (react!
(if (< @a2 1) @b @c))]
(is (= @res (+ 2 @a)))
(is (= @b-changed 1))
Expand Down Expand Up @@ -144,7 +144,7 @@
c (reaction (dec @a))
d (reaction (str @b))
res (rv/atom 0)
cs (run!
cs (react!
(reset! res @d))]
(is (= @res "1"))
(dispose cs))
Expand All @@ -154,17 +154,17 @@
a (r/cursor a-base [:a])
b (reaction (inc @a))
c (reaction (dec @a))
d (run! [@b @c])]
d (react! [@b @c])]
(is (= @d [1 -1]))
(dispose d))
(let [a-base (rv/atom 0)
a (r/cursor a-base [])
b (reaction (inc @a))
c (reaction (dec @a))
d (run! [@b @c])
d (react! [@b @c])
res (rv/atom 0)]
(is (= @d [1 -1]))
(let [e (run! (reset! res @d))]
(let [e (react! (reset! res @d))]
(is (= @res [1 -1]))
(dispose e))
(dispose d))
Expand Down
20 changes: 10 additions & 10 deletions test/reagenttest/testratom.cljs
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
(ns reagenttest.testratom
(:require [cljs.test :as t :refer-macros [is deftest testing]]
[reagent.ratom :as rv :refer-macros [run! reaction]]
[reagent.ratom :as rv :refer-macros [react! reaction]]
[reagent.debug :refer-macros [dbg]]
[reagent.core :as r]))

Expand All @@ -15,7 +15,7 @@
(dbg "ratom-perf")
(let [a (rv/atom 0)
mid (reaction (inc @a))
res (run!
res (react!
(inc @mid))]
(time (dotimes [x 100000]
(swap! a inc)))
Expand All @@ -34,7 +34,7 @@
res (reaction
(swap! count inc)
@sv @c2 @comp)
const (run!
const (react!
(reset! out @res))]
(is (= @count 1) "constrain ran")
(is (= @out 2))
Expand Down Expand Up @@ -69,7 +69,7 @@
(let [runs (running)]
(let [!counter (rv/atom 0)
!signal (rv/atom "All I do is change")
co (run!
co (react!
;;when I change...
@!signal
;;update the counter
Expand Down Expand Up @@ -102,7 +102,7 @@
c (reaction
(swap! c-changed inc)
(+ 10 @a2))
res (run!
res (react!
(if (< @a2 1) @b @c))]
(is (= @res (+ 2 @a)))
(is (= @b-changed 1))
Expand Down Expand Up @@ -141,7 +141,7 @@
c (reaction (dec @a))
d (reaction (str @b))
res (rv/atom 0)
cs (run!
cs (react!
(reset! res @d))]
(is (= @res "1"))
(dispose cs))
Expand All @@ -150,16 +150,16 @@
(let [a (rv/atom 0)
b (reaction (inc @a))
c (reaction (dec @a))
d (run! [@b @c])]
d (react! [@b @c])]
(is (= @d [1 -1]))
(dispose d))
(let [a (rv/atom 0)
b (reaction (inc @a))
c (reaction (dec @a))
d (run! [@b @c])
d (react! [@b @c])
res (rv/atom 0)]
(is (= @d [1 -1]))
(let [e (run! (reset! res @d))]
(let [e (react! (reset! res @d))]
(is (= @res [1 -1]))
(dispose e))
(dispose d))
Expand Down Expand Up @@ -243,7 +243,7 @@
;; a (rv/atom false)
;; catch-count (atom 0)
;; b (reaction (if @a (throw {})))
;; c (run! (try @b (catch js/Object e
;; c (react! (try @b (catch js/Object e
;; (swap! catch-count inc))))]
;; (is (= @catch-count 0))
;; (reset! a false)
Expand Down

0 comments on commit 2ed9291

Please sign in to comment.