[php-src] master: standard/filestats.c: use RETURN_BOOL() when possible (#22876)
Gina Peter Banyard via GitHub <[email protected]> Sat, 25 Jul 2026 10:37:03 +0000
| Newsgroups | gmane.comp.php.cvs.general |
|---|---|
| Message-ID | <[email protected]> |
Author: Gina Peter Banyard (Girgias)
Committer: GitHub (web-flow)
Pusher: Girgias
Date: 2026-07-25T11:37:00+01:00
Commit: https://github.com/php/php-src/commit/bf46492421b4b74247285a591d7251b53e0a7191
Raw diff: https://github.com/php/php-src/commit/bf46492421b4b74247285a591d7251b53e0a7191.diff
standard/filestats.c: use RETURN_BOOL() when possible (#22876)
Changed paths:
M ext/standard/filestat.c
Diff:
diff --git a/ext/standard/filestat.c b/ext/standard/filestat.c
index f7bda3c66bab..7dcfecf1f872 100644
--- a/ext/standard/filestat.c
+++ b/ext/standard/filestat.c
@@ -339,11 +339,7 @@ static void php_do_chgrp(INTERNAL_FUNCTION_PARAMETERS, int do_lchgrp) /* {{{ */
value = &group_long;
}
- if(wrapper->wops->stream_metadata(wrapper, filename, option, value, NULL)) {
- RETURN_TRUE;
- } else {
- RETURN_FALSE;
- }
+ RETURN_BOOL(wrapper->wops->stream_metadata(wrapper, filename, option, value, NULL));
} else {
#ifndef PHP_WIN32
/* On Windows, we expect regular chgrp to fail silently by default */
@@ -480,11 +476,7 @@ static void php_do_chown(INTERNAL_FUNCTION_PARAMETERS, int do_lchown) /* {{{ */
value = &user_long;
}
- if(wrapper->wops->stream_metadata(wrapper, filename, option, value, NULL)) {
- RETURN_TRUE;
- } else {
- RETURN_FALSE;
- }
+ RETURN_BOOL(wrapper->wops->stream_metadata(wrapper, filename, option, value, NULL));
} else {
#ifndef PHP_WIN32
/* On Windows, we expect regular chown to fail silently by default */
@@ -638,11 +630,7 @@ PHP_FUNCTION(touch)
wrapper = php_stream_locate_url_wrapper(filename, NULL, 0);
if(wrapper != &php_plain_files_wrapper || strncasecmp("file://", filename, 7) == 0) {
if(wrapper && wrapper->wops->stream_metadata) {
- if(wrapper->wops->stream_metadata(wrapper, filename, PHP_STREAM_META_TOUCH, newtime, NULL)) {
- RETURN_TRUE;
- } else {
- RETURN_FALSE;
- }
+ RETURN_BOOL(wrapper->wops->stream_metadata(wrapper, filename, PHP_STREAM_META_TOUCH, newtime, NULL));
} else {
php_stream *stream;
if(!filetime_is_null || !fileatime_is_null) {