Many other reasons that setvbuf can fail than "wrong arguments", returning

false should be enough

diff --git a/ext/standard/file.c b/ext/standard/file.c
index 5b4f92749caac5a0935bbedc2353abe0764bc4a8..0bfd0da89d3690a0d6ab5027d86e1c4ae9b2b71e 100644 (file)
--- a/ext/standard/file.c
+++ b/ext/standard/file.c
@@ -1391,9 +1391,6 @@ PHP_FUNCTION(set_file_buffer)
                ret = setvbuf((FILE *) what, NULL, _IOFBF, buff);
        }
 
-       if (ret) {
-               php_error(E_WARNING, "Illegal value for set_file_buffer()");
-       }
        RETURN_LONG(ret);
 }
 /* }}} */