diff options
author | Christoph M. Becker <cmbecker69@gmx.de> | 2019-08-16 17:46:46 +0200 |
---|---|---|
committer | Christoph M. Becker <cmbecker69@gmx.de> | 2019-08-16 17:54:11 +0200 |
commit | 2d1f371871ef5afee6b2d683c760b25c8b147c66 (patch) | |
tree | a3870990fcd305b0e05c900ca26416d2a5840505 /ext/bz2 | |
parent | 1f49b1df507787833cef12b63c2fcd022453c19e (diff) | |
download | php-git-2d1f371871ef5afee6b2d683c760b25c8b147c66.tar.gz |
Don't explicitly set return value on ZPP failure
Failing ZPP throws as of PHP 8.0.0, so explicitly setting a return
value is useless, and also slightly confusing.
Diffstat (limited to 'ext/bz2')
-rw-r--r-- | ext/bz2/bz2.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ext/bz2/bz2.c b/ext/bz2/bz2.c index bf706f6449..b4079df5d5 100644 --- a/ext/bz2/bz2.c +++ b/ext/bz2/bz2.c @@ -335,7 +335,7 @@ static PHP_FUNCTION(bzread) zend_string *data; if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS(), "r|l", &bz, &len)) { - RETURN_FALSE; + return; } php_stream_from_zval(stream, bz); @@ -535,7 +535,7 @@ static PHP_FUNCTION(bzdecompress) bz_stream bzs; if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS(), "s|l", &source, &source_len, &small)) { - RETURN_FALSE; + return; } bzs.bzalloc = NULL; |