Skip to content

Commit 9c9b4d9

Browse files
committed
Use RETURN_THROWS();
1 parent 158993b commit 9c9b4d9

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

ext/standard/file.c

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -352,7 +352,7 @@ PHP_FUNCTION(flock)
352352
act = operation & 3;
353353
if (act < 1 || act > 3) {
354354
zend_value_error("Illegal operation argument");
355-
return;
355+
RETURN_THROWS();
356356
}
357357

358358
if (wouldblock) {
@@ -746,7 +746,7 @@ PHP_FUNCTION(file)
746746

747747
if (flags < 0 || flags > (PHP_FILE_USE_INCLUDE_PATH | PHP_FILE_IGNORE_NEW_LINES | PHP_FILE_SKIP_EMPTY_LINES | PHP_FILE_NO_DEFAULT_CONTEXT)) {
748748
zend_value_error("'" ZEND_LONG_FMT "' flag is not supported", flags);
749-
return;
749+
RETURN_THROWS();
750750
}
751751

752752
use_include_path = flags & PHP_FILE_USE_INCLUDE_PATH;
@@ -1041,7 +1041,7 @@ PHPAPI PHP_FUNCTION(fgets)
10411041
} else if (argc > 1) {
10421042
if (len <= 0) {
10431043
zend_value_error("Length parameter must be greater than 0");
1044-
return;
1044+
RETURN_THROWS();
10451045
}
10461046

10471047
str = zend_string_alloc(len, 0);
@@ -1496,7 +1496,7 @@ PHP_NAMED_FUNCTION(php_if_ftruncate)
14961496

14971497
if (size < 0) {
14981498
zend_value_error("Negative size is not supported");
1499-
return;
1499+
RETURN_THROWS();
15001500
}
15011501

15021502
PHP_STREAM_TO_ZVAL(stream, fp);
@@ -1751,7 +1751,7 @@ PHPAPI PHP_FUNCTION(fread)
17511751

17521752
if (len <= 0) {
17531753
zend_value_error("Length parameter must be greater than 0");
1754-
return;
1754+
RETURN_THROWS();
17551755
}
17561756

17571757
str = php_stream_read_to_str(stream, len);
@@ -1830,7 +1830,7 @@ PHP_FUNCTION(fputcsv)
18301830
/* Make sure that there is at least one character in string */
18311831
if (delimiter_str_len < 1) {
18321832
zend_value_error("delimiter must be a character");
1833-
return;
1833+
RETURN_THROWS();
18341834
} else if (delimiter_str_len > 1) {
18351835
php_error_docref(NULL, E_NOTICE, "delimiter must be a single character");
18361836
}
@@ -1842,7 +1842,7 @@ PHP_FUNCTION(fputcsv)
18421842
if (enclosure_str != NULL) {
18431843
if (enclosure_str_len < 1) {
18441844
zend_value_error("enclosure must be a character");
1845-
return;
1845+
RETURN_THROWS();
18461846
} else if (enclosure_str_len > 1) {
18471847
php_error_docref(NULL, E_NOTICE, "enclosure must be a single character");
18481848
}
@@ -1968,7 +1968,7 @@ PHP_FUNCTION(fgetcsv)
19681968
/* Make sure that there is at least one character in string */
19691969
if (delimiter_str_len < 1) {
19701970
zend_value_error("delimiter must be a character");
1971-
return;
1971+
RETURN_THROWS();
19721972
} else if (delimiter_str_len > 1) {
19731973
php_error_docref(NULL, E_NOTICE, "delimiter must be a single character");
19741974
}
@@ -1980,7 +1980,7 @@ PHP_FUNCTION(fgetcsv)
19801980
if (enclosure_str != NULL) {
19811981
if (enclosure_str_len < 1) {
19821982
zend_value_error("enclosure must be a character");
1983-
return;
1983+
RETURN_THROWS();
19841984
} else if (enclosure_str_len > 1) {
19851985
php_error_docref(NULL, E_NOTICE, "enclosure must be a single character");
19861986
}
@@ -2005,7 +2005,7 @@ PHP_FUNCTION(fgetcsv)
20052005
len = zval_get_long(len_zv);
20062006
if (len < 0) {
20072007
zend_value_error("Length parameter may not be negative");
2008-
return;
2008+
RETURN_THROWS();
20092009
} else if (len == 0) {
20102010
len = -1;
20112011
}

0 commit comments

Comments
 (0)