Re: [PATCH] Add cereal + sparsehash

[Ricardo Wurmus]

Andreas Enge <address@hidden> writes:

> On Thu, Nov 26, 2015 at 01:53:25PM +0100, Ricardo Wurmus wrote:
>> The second patch adds “sparsehash” to the “crypto” module.  It’s
>> probably not the best module for this package — can you suggest any
>> other location for a hash table library?
>
> If it is not a cryptographically secure hash, it should go anywhere, but
> not to the cryptography module. Both are called by the same name in English,
> but are completely different things. Could you move it somewhere else?

I’d like to do this, but I’m pretty bad at naming things.  How about a
new module “datastructures.scm” where this could comfortably fit in?  Or
is this too vague?  Or should it go in the equally vague “code.scm”
(which actually is for tools operating on source code)?

~~ Ricardo