search

gyson / blex

Fast Bloom filter with concurrent accessibility, powered by :atomics module.
MIT License
38 stars 2 forks source link

Use `Kernel.ceil/1` instead of `Float.ceil/1` + `Kernel.trunc/1` #1

Closed preciz closed 5 years ago

preciz commented 5 years ago

Just a small edit to make code more terse.

gyson commented 5 years ago

👍

Kernel.ceil/1 is only available since Elixir 1.8.0, we would need to update Elixir version requirement here as well.

preciz commented 5 years ago

I did a quick glance over Kernel.ceil/1 in the source but haven't noticed, thanks, maybe then merge later.

gyson commented 5 years ago

Nice! I will merge this into master first. May create a new release later (probably sometime after Elixir 1.9 official available).