brandonbloom / fipp

Fast Idiomatic Pretty Printer for Clojure
525 stars 44 forks source link

remove cljs symbol conflict #36

Closed aiba closed 8 years ago

aiba commented 8 years ago

When I use fipp from clojurescript, I get a compile warning:

WARNING: char? already refers to: cljs.core/char? being replaced by: fipp.visit/char? at line 34 target/cljsbuild/dev/out/fipp/visit.cljc

This PR fixes that.

brandonbloom commented 8 years ago

LGTM.

The char? predicate is new in CLJS, but it's sorta goofy because there is no actual char type, so this seems like the right thing to do to me.

brandonbloom commented 8 years ago

Deployed v0.6.5