-
Notifications
You must be signed in to change notification settings - Fork 24
Adding support to the 64bit version of xxhash #2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Conversation
* Adding support for the 64bit version of the xxhash library
Adding support to the 64bit version of xxhash
Thank you for adding this! Was looking for a 64-bit implementation. |
I'd like to have a php function call for createing a xxhash from a file not from a string. To handle with strings is very poor for very large data. Like md5_file (). Thanks a lot. GeThi255 |
Can you do xxhash(file_get_contents('file'))? At work we do this with PDFs up to 20 MB or so and it works fine. |
Thank you very much for the solution. But this kills the performance. 1st try: 2nd try: 3rd try: My largest file has 143 GB. It's a VM image. |
And the max file size is limited by the RAM. Tried with 17 GB file: |
Comments prevent buildconf from correctly generating configure.js.
Remove comments to fix configure.js
Zend RETURN_STRING function was changed
…ate-php-7 # Conflicts: # README.md # config.w32 # php_xxhash.c # php_xxhash.h # xxhash.c # xxhash.h
…rt unsigned long)
# Conflicts: # README.md
I needed the 64 bit version, I assumed other people will need it too. The rest of the details are in the README.md file.
General note:
Since PHP doesn't support
unsigned long
values, thexxhash64()
function can return negative values (this is the same case with the JAVA library). In case you are working in multiple platforms you can use thexxhash64Unsigned()
version I added, or casting the results in other platforms fromunsigned long
tolong