search

clj-commons / byte-streams

A Rosetta stone for JVM byte representations
417 stars 33 forks source link

Copy namespaces into `clj-commons` and update `primitive-math` dependency #54

Closed skynet-gh closed 2 years ago

skynet-gh commented 2 years ago

Fixes issues with single segment namespaces. Keep existing namespaces for compatibility.

Updates primitive-math dependency which was updated similarly in https://github.com/clj-commons/primitive-math/pull/14\

Tests:

$ lein test
Compiling 3 source files to /home/skynet/git/byte-streams/target/classes

lein test byte-streams-simple-check
{:result true, :num-tests 10000, :seed 1649107149557, :time-elapsed-ms 594, :test-var "equivalent-comparison"}

lein test byte-streams-test

lein test clj-commons.byte-streams-simple-check
{:result true, :num-tests 10000, :seed 1649107189494, :time-elapsed-ms 372, :test-var "equivalent-comparison"}

lein test clj-commons.byte-streams-test

lein test pushback-stream-test

Ran 15 tests containing 1003 assertions.
0 failures, 0 errors
skynet-gh commented 2 years ago

@KingMob sure no problem, I think I've updated all the namespaces as you describe.

skynet-gh commented 2 years ago

@KingMob whoops! fixed

KingMob commented 2 years ago

Sweet! I’ll double-check this weekend when I’m free, but I think that’s it.