search

rtfeldman / elm-css

Typed CSS in Elm.
https://package.elm-lang.org/packages/rtfeldman/elm-css/latest
BSD 3-Clause "New" or "Revised" License
1.23k stars 196 forks source link

Replace fnv with murmur3 #556

Closed robinheghan closed 2 years ago

robinheghan commented 2 years ago

After talking to Richard, we agreed to hold back the hash-function change. FNV is a slightly worse hash than Murmur3, and elm-css doesn't have a way to handle hash collisions. It's safer to stick with Murmur3, for now.