@lee thanks for the headsup - yeah, closed those
@henryw374 open to that, add the patch to the issue
Corner case with and
/ or
: https://clojure.atlassian.net/browse/CLJS-3309
@mfikes thanks! interesting no other tests failed that look like this
Coal mine found a handful...
all the same and only with or?
Actually, I hadn't surveyed across all of them to see if there are any distinct things outside of the gensym issue... let me take a peek.
Here is where I found them FWIW https://travis-ci.org/github/mfikes/coal-mine/builds/771413091
all seem to involve or
They all look like and
and or
gensym failures that all fall into the same "category"
can you include an and
repro in the issue?
Here is an and
one https://travis-ci.org/github/mfikes/coal-mine/jobs/771413094#L635
would be a good unit test
Yeah, I'll make a minimal from that one I just linked...
thanks!
@mfikes hrm it appears these all involve loop?
The and
one I'm making a minimal for involves a for
it appears.
((fn [s] (for [e (tree-seq sequential? seq s) :when (and (sequential? e) (every? (fn [x] (not (sequential? x))) e))] e)) [[1 2] [[3 4] [[[[5 6]]]]]])
^ Trying to make that one a little smaller
ah they are all similar