get rid of HashCodeComparator and all its usages

[mohrm]

This addresses both #133 and #88

[msridhar]

So the HashCodeComparator cods is really old and I can't tell from version control history why it was introduced. I'm guessing that at some point, module entries were being returned in a non-deterministic order, and this was an attempt to iterate through them deterministically. But I agree that it's not the right way to do it, at least not anymore. Seems like Julian is also OK with this change, so merging. Thanks for finding this!