It's redundant calling hash.hash_fn for example.  Standardize on fn
and data as names (private conflicts with C++).