[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Add cereal + sparsehash
From: |
Ludovic Courtès |
Subject: |
Re: [PATCH] Add cereal + sparsehash |
Date: |
Tue, 01 Dec 2015 22:55:01 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
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’.