[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: Add xxhash.
From: |
David Thompson |
Subject: |
Re: [PATCH] gnu: Add xxhash. |
Date: |
Tue, 28 Apr 2015 11:36:29 -0400 |
User-agent: |
Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (x86_64-pc-linux-gnu) |
Andreas Enge <address@hidden> writes:
> On Sun, Apr 26, 2015 at 05:53:32PM -0400, David Thompson wrote:
>> * gnu/packages/xxhash.scm: New file.
>
> I am usually not in favour of adding new files for small utilities. Could it
> go somewhere else?
I don't like it either, but there's no existing module that makes
sense. Can anyone think of one? It this tool created cryptographic
hashes then I would have put it in crypto.scm.
>
>> + (synopsis "Extremely fast non-cryptographic hash algorithm")
>> + (description
>> + "xxHash is an extremely fast hash algorithm, running at RAM speed
>> limits.
>> +It successfully passes the SMHasher test suite evaluating hash quality.")
>
> Maybe drop the marketing parts "extremely fast"?
Okay.
--
David Thompson
Web Developer - Free Software Foundation - http://fsf.org
GPG Key: 0FF1D807
Support the FSF: https://fsf.org/donate