search

akhudek / fast-zip

fast-zip
85 stars 16 forks source link

Fix seq-zip to avoid spurious nils #3

Open michalmarczyk opened 10 years ago

michalmarczyk commented 10 years ago

See CLJ-1317:

http://dev.clojure.org/jira/browse/CLJ-1317

Credit for discovering the problem goes to Lee Spector. More comments on the Clojure ticket.

Note that this is a completely fresh issue, with no comments from Clojure/core on the patch.

akhudek commented 10 years ago

Thanks!

I'd like to wait for Clojure/core's response as I'd like this library to have matching semantics to clojure's zip (including bugs). Assuming this is included in clojure 1.6, I'll make a new release matching that with appropriate documentation.

michalmarczyk commented 10 years ago

Sure, that makes sense. Users can just include this seq-zip implementation anyway.