Re: [PATCH] Add cereal + sparsehash

[Ludovic Courtès]

Ricardo Wurmus <address@hidden> skribis:

> 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)?

I see there’s cityhash in textutils.scm and gperf in its own module.
Maybe textutils.scm?  Or next to cereal?

Ludo’.