RibbonFilter / ribbonGo / 23252781544

94%

main: 94%

Pull #39

github

fix: delegate Ribbon.Contains to filter.contains instead of calling xxh3 directly

Ribbon.Contains was directly calling xxh3.HashString(key) and passing
the result to f.containsHash(), bypassing the hasher abstraction that
filter.contains() already encapsulates.

This couples the public API to a specific hash implementation. If
keyHashString() is ever changed (e.g. adding a salt, switching hash
functions), Ribbon.Contains would silently diverge from Build,
producing incorrect query results.

Delegate to f.contains(key) instead, which correctly goes through
the hasher abstraction. No performance impact — keyHashString is the
same xxh3.HashString call today, and the method is inlineable.

Fixes #37

1 of 1 new or added line in 1 file covered. (100.0%)

875 of 929 relevant lines covered (94.19%)

4029277.55 hits per line