Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:112120
MIME-Version: 1.0
References: <CAJmy8YFJZ7nfjYPZX2vzhxp+F7tLb1eDiVNnWqOcH+ktR0i9sg@mail.gmail.com>
 <CAF+90c9G=jtpogVGN4fy94jv232WZwzghyErkQTjxHDQeYY3xw@mail.gmail.com>
 <CAJmy8YHzowbxhGSzaRQWJOyt5hBrOCkM=RpWweOkdx2NGRzcQQ@mail.gmail.com> <33788573-5cf1-0409-5cfc-d090b96b4db8@gmx.de>
In-Reply-To: <33788573-5cf1-0409-5cfc-d090b96b4db8@gmx.de>
Date: Mon, 26 Oct 2020 13:08:36 +0100
Message-ID: <CAJmy8YFPokbdugXWWRJZcxt6FvRRgj9vqKCvnGb33YEDnWaUYQ@mail.gmail.com>
To: PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Subject: Re: [PHP-DEV] why is non-crypto hashes excluded from/ext/hash/bench.php
 ?
From: divinity76@gmail.com (Hans Henrik Bergan)

@Christoph M. Becker

haha that's probably it, i guess that long ago PHP didn't have any
non-crypto hash algos,
one of the results read "256MB Windows 2000"
when was the last time someone rolled a 256MB ram windows 2000?
(am rolling some 128MB ram VPS's, but rolling Windows with such
amounts of ram would be insane today)

nevermind then, thanks for pointing it out (also seems Nikita Popov
just removed the old results from master)


.. another thing, imo /ext/hash/bench.php should hash a constant size,
not some size that change every time the code changes, in my opinion
$data = file_get_contents(__FILE__);
should be replaced with something like
$data = str_repeat("\x00", 2 * 1024);

- the old size of bench.php (before Nikita's edit) was 2240 bytes, and
2*1024 is pretty close, 2048 bytes
- if you for some reason want random bytes instead of 0, i guess
random_bytes() can be used instead, but i don't think it should make
any difference in the benchmarked parts