Fork me on GitHub
#nrepl
<
2019-02-17
>
dominicm21:02:52

I have a bit of a dilemma.

dominicm21:02:25

Fireplace doesn't require cider-nrepl. But the current pprint default is cider.fipp because that's fine: it would only be interpreted of cider was loaded.

dominicm21:02:51

I suppose fireplace will have to default to clojure.pprint, which is unfortunately slow and prone to StackOverflows

Michael Griffiths22:02:55

It’s also a shame how clojure.pprint rebinds *out*, but on the other hand it pretty prints some things that fipp does not (e.g. Datomic entities; see https://github.com/clojure-emacs/cider/issues/1505)

dominicm22:02:49

That's a benefit of the native integration.

dominicm22:02:26

I wonder if fipp could use pprint instead of prn when unsure. It would hurt performance though.