kjdev / php-ext-zstd

Zstd Extension for PHP
MIT License
201 stars 27 forks source link

Warning: zstd_compress: compression level (100) must be within 1..22 or smaller then 0 #29

Closed glensc closed 3 years ago

glensc commented 4 years ago

For me this test fails for php 5.3-5.6, 7.0-7.3, passes on 7.4

TEST 10/24 [tests/008.phpt]
========DIFF========
028+ Warning: zstd_compress: compression level (100) must be within 1..22 or smaller then 0 in /tmp/B.lZPARi/BUILD/php73-pecl-zstd-0.8.0/tests/008.php on line 6
029+ 100 -- 0 --
030+ Warning: zstd_uncompress: it was not compressed by zstd in /tmp/B.lZPARi/BUILD/php73-pecl-zstd-0.8.0/tests/008.php on line 8
031+ false
032+ -1 -- 2674 -- true
033+ 0 -- 3547 --
034+ Warning: zstd_uncompress: it was not compressed by zstd in /tmp/B.lZPARi/BUILD/php73-pecl-zstd-0.8.0/tests/008.php on line 8
035+ false
028- Warning: zstd_compress: compression level (100) must be within 1..22 in %s on line %d
029- 100 -- 0 -- false
030-
031- Warning: zstd_compress: compression level (-1) must be within 1..22 in %s on line %d
032- -1 -- 0 -- false
033- 0 -- 3547 -- false

Build logs:

kjdev commented 3 years ago

409e920 Fixed a problem with libzstd's version output.