This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2018-01-17
Channels
- # adventofcode (2)
- # beginners (153)
- # cider (14)
- # clara (9)
- # cljs-dev (8)
- # cljsjs (1)
- # cljsrn (4)
- # clojure (124)
- # clojure-dev (9)
- # clojure-france (18)
- # clojure-greece (22)
- # clojure-italy (11)
- # clojure-nlp (5)
- # clojure-russia (9)
- # clojure-spec (21)
- # clojure-uk (40)
- # clojurescript (82)
- # core-async (12)
- # cursive (3)
- # data-science (2)
- # datomic (225)
- # devcards (8)
- # docs (2)
- # duct (1)
- # emacs (18)
- # figwheel (2)
- # fulcro (117)
- # graphql (13)
- # hoplon (10)
- # jobs (7)
- # jobs-discuss (7)
- # keechma (8)
- # leiningen (4)
- # off-topic (16)
- # om (2)
- # om-next (3)
- # perun (11)
- # precept (4)
- # re-frame (24)
- # reagent (2)
- # remote-jobs (8)
- # ring (2)
- # ring-swagger (9)
- # rum (42)
- # shadow-cljs (8)
- # spacemacs (3)
- # specter (7)
- # uncomplicate (10)
- # unrepl (58)
- # yada (9)
^ FWIW, https://dev.clojure.org/jira/browse/TRDR-50 (looks like a 3 year-old Clojure->ClojureScript tools reader porting bug that nobody has noticed until now)
@mfikes thanks for helping with the issue! I compile with "exit-on-warning" always on and I could not miss it I guess 😉
the (easy) patch solves
@richiardiandrea With the patch in https://dev.clojure.org/jira/browse/CLJS-2473, you get this in JVM ClojureScript
cljs.user=> (int \uD7ff)
WARNING: cljs.core/bit-or, all arguments must be numbers, got [string number] instead. at line 1 <cljs repl>
0
and we also get helpful warnings when running lein test
:
WARNING: cljs.core/bit-or, all arguments must be numbers, got [string number] instead. at line 117 /var/folders/gx/nymj3l7x4zq3gxb97v2zwzb40000gn/T/loader-test-out/cljs/tools/reader/edn.cljs
WARNING: cljs.core/bit-or, all arguments must be numbers, got [string number] instead. at line 118 /var/folders/gx/nymj3l7x4zq3gxb97v2zwzb40000gn/T/loader-test-out/cljs/tools/reader/edn.cljs
Oh so cljs is affected too, well glad we solved
Oh ok, this is a new patch ;)