Closed GoogleCodeExporter closed 8 years ago
also -d# should be *de*compression function -- typo in usage_advanced().
Original comment by ejkr...@gmail.com
on 1 Apr 2014 at 5:53
Thanks, clear bug report and straightforward solution.
It'll be part of next release candidate.
Original comment by yann.col...@gmail.com
on 6 Apr 2014 at 6:58
I couldn't find the -d# option within usage_advanced(). Are you sure about this
one ?
Original comment by yann.col...@gmail.com
on 6 Apr 2014 at 7:56
You can test a release candidate version, correcting your (1st) reported bug at
:
https://github.com/Cyan4973/lz4/tree/beta
Original comment by yann.col...@gmail.com
on 6 Apr 2014 at 8:27
When I run my local "fullbench" this is what I get:
kruus@borg:~/ssddup-ubd/cache/dev/lz4-r116/programs$ ./fullbench -h
*** LZ4 speed analyzer r116 64-bits, by Yann Collet [mods Erik Kruus] (Apr 2
2014) ***
Usage :
./fullbench [arg] file1 file2 ... fileX
Arguments :
-c : compression tests only
-d : decompression tests only
-H/-h : Help (this text + advanced options)
Advanced options :
-c# : test only compression function # [0-@]
^
(oh, I had more than 9 versions in my tests, ignore this)
-d# : test only decompression function # [0-5] <---- *** add "de" ***
^^
-i# : iteration loops [1-9](default : 6)
-B# : Block size [4-7](default : 7)
(... my version continues printing -c#/-d# names, not important)
The corrected typo in fullbench.c is:
DISPLAY( " -d# : test only decompression function # [%c-%c]\n",
MINDECOMPRESSIONCHAR, MAXDECOMPRESSIONCHAR);
Original comment by ejkr...@gmail.com
on 7 Apr 2014 at 2:02
Yes, you are totally correct.
The issue is fixed on the beta branch of github mirror now :
https://github.com/Cyan4973/lz4/tree/beta
Original comment by yann.col...@gmail.com
on 8 Apr 2014 at 11:04
Fixed into r117
Original comment by yann.col...@gmail.com
on 22 Apr 2014 at 11:07
Original issue reported on code.google.com by
ejkr...@gmail.com
on 1 Apr 2014 at 5:05