]> granicus.if.org Git - php/commitdiff
Update fileinfo parameter names
authorNikita Popov <nikita.ppv@gmail.com>
Thu, 24 Sep 2020 13:37:10 +0000 (15:37 +0200)
committerNikita Popov <nikita.ppv@gmail.com>
Thu, 24 Sep 2020 16:36:36 +0000 (18:36 +0200)
Closes GH-6206.

ext/fileinfo/fileinfo.stub.php
ext/fileinfo/fileinfo_arginfo.h
ext/fileinfo/tests/finfo_open_001.phpt
ext/fileinfo/tests/finfo_open_error.phpt

index a951e573dcd07eb305a551e10e59fb5827d50c72..9355a96a2b192bfad2b20ec5110868f9a743c7ec 100644 (file)
@@ -5,31 +5,31 @@
 class finfo
 {
     /** @alias finfo_open */
-    public function __construct(int $options = FILEINFO_NONE, string $arg = "") {}
+    public function __construct(int $flags = FILEINFO_NONE, string $magic_database = "") {}
 
     /**
      * @param resource|null $context
      * @return string|false
      * @alias finfo_file
      */
-    public function file(string $file_name, int $options = FILEINFO_NONE, $context = null) {}
+    public function file(string $filename, int $flags = FILEINFO_NONE, $context = null) {}
 
     /**
      * @param resource|null $context
      * @return string|false
      * @alias finfo_buffer
      */
-    public function buffer(string $string, int $options = FILEINFO_NONE, $context = null) {}
+    public function buffer(string $string, int $flags = FILEINFO_NONE, $context = null) {}
 
     /**
      * @return bool
      * @alias finfo_set_flags
      */
-    public function set_flags(int $options) {}
+    public function set_flags(int $flags) {}
 }
 
 /** @return resource|false */
-function finfo_open(int $options = FILEINFO_NONE, string $arg = "") {}
+function finfo_open(int $flags = FILEINFO_NONE, string $magic_database = "") {}
 
 /**
  * @param resource $finfo
@@ -39,19 +39,19 @@ function finfo_close($finfo): bool {}
 /**
  * @param resource $finfo
  */
-function finfo_set_flags($finfo, int $options): bool {}
+function finfo_set_flags($finfo, int $flags): bool {}
 
 /**
  * @param resource $finfo
  * @param resource|null $context
  */
-function finfo_file($finfo, string $file_name, int $options = FILEINFO_NONE, $context = null): string|false {}
+function finfo_file($finfo, string $filename, int $flags = FILEINFO_NONE, $context = null): string|false {}
 
 /**
  * @param resource $finfo
  * @param resource|null $context
  */
-function finfo_buffer($finfo, string $string, int $options = FILEINFO_NONE, $context = null): string|false {}
+function finfo_buffer($finfo, string $string, int $flags = FILEINFO_NONE, $context = null): string|false {}
 
 /**
  * @param resource|string $filename
index 95fe4fc98bf74c0cc0ae1ac10c65f6df2f09baab..80850a6502e6117819a29f96eebae7efa6c6f4e9 100644 (file)
@@ -1,9 +1,9 @@
 /* This is a generated file, edit the .stub.php file instead.
- * Stub hash: b8bfd01bd5a9a181550160e39c50a8cc047f15cd */
+ * Stub hash: be858509df27550b51d8a7a51a3629eceb6d0aa6 */
 
 ZEND_BEGIN_ARG_INFO_EX(arginfo_finfo_open, 0, 0, 0)
-       ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, options, IS_LONG, 0, "FILEINFO_NONE")
-       ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, arg, IS_STRING, 0, "\"\"")
+       ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, flags, IS_LONG, 0, "FILEINFO_NONE")
+       ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, magic_database, IS_STRING, 0, "\"\"")
 ZEND_END_ARG_INFO()
 
 ZEND_BEGIN_ARG_WITH_RETURN_TYPE_INFO_EX(arginfo_finfo_close, 0, 1, _IS_BOOL, 0)
@@ -12,20 +12,20 @@ ZEND_END_ARG_INFO()
 
 ZEND_BEGIN_ARG_WITH_RETURN_TYPE_INFO_EX(arginfo_finfo_set_flags, 0, 2, _IS_BOOL, 0)
        ZEND_ARG_INFO(0, finfo)
-       ZEND_ARG_TYPE_INFO(0, options, IS_LONG, 0)
+       ZEND_ARG_TYPE_INFO(0, flags, IS_LONG, 0)
 ZEND_END_ARG_INFO()
 
 ZEND_BEGIN_ARG_WITH_RETURN_TYPE_MASK_EX(arginfo_finfo_file, 0, 2, MAY_BE_STRING|MAY_BE_FALSE)
        ZEND_ARG_INFO(0, finfo)
-       ZEND_ARG_TYPE_INFO(0, file_name, IS_STRING, 0)
-       ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, options, IS_LONG, 0, "FILEINFO_NONE")
+       ZEND_ARG_TYPE_INFO(0, filename, IS_STRING, 0)
+       ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, flags, IS_LONG, 0, "FILEINFO_NONE")
        ZEND_ARG_INFO_WITH_DEFAULT_VALUE(0, context, "null")
 ZEND_END_ARG_INFO()
 
 ZEND_BEGIN_ARG_WITH_RETURN_TYPE_MASK_EX(arginfo_finfo_buffer, 0, 2, MAY_BE_STRING|MAY_BE_FALSE)
        ZEND_ARG_INFO(0, finfo)
        ZEND_ARG_TYPE_INFO(0, string, IS_STRING, 0)
-       ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, options, IS_LONG, 0, "FILEINFO_NONE")
+       ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, flags, IS_LONG, 0, "FILEINFO_NONE")
        ZEND_ARG_INFO_WITH_DEFAULT_VALUE(0, context, "null")
 ZEND_END_ARG_INFO()
 
@@ -36,19 +36,19 @@ ZEND_END_ARG_INFO()
 #define arginfo_class_finfo___construct arginfo_finfo_open
 
 ZEND_BEGIN_ARG_INFO_EX(arginfo_class_finfo_file, 0, 0, 1)
-       ZEND_ARG_TYPE_INFO(0, file_name, IS_STRING, 0)
-       ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, options, IS_LONG, 0, "FILEINFO_NONE")
+       ZEND_ARG_TYPE_INFO(0, filename, IS_STRING, 0)
+       ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, flags, IS_LONG, 0, "FILEINFO_NONE")
        ZEND_ARG_INFO_WITH_DEFAULT_VALUE(0, context, "null")
 ZEND_END_ARG_INFO()
 
 ZEND_BEGIN_ARG_INFO_EX(arginfo_class_finfo_buffer, 0, 0, 1)
        ZEND_ARG_TYPE_INFO(0, string, IS_STRING, 0)
-       ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, options, IS_LONG, 0, "FILEINFO_NONE")
+       ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, flags, IS_LONG, 0, "FILEINFO_NONE")
        ZEND_ARG_INFO_WITH_DEFAULT_VALUE(0, context, "null")
 ZEND_END_ARG_INFO()
 
 ZEND_BEGIN_ARG_INFO_EX(arginfo_class_finfo_set_flags, 0, 0, 1)
-       ZEND_ARG_TYPE_INFO(0, options, IS_LONG, 0)
+       ZEND_ARG_TYPE_INFO(0, flags, IS_LONG, 0)
 ZEND_END_ARG_INFO()
 
 
index 1b1a1a690018fa89dd3d8b7231cb203d39c22165..b2285dd35c4dfbc8ce554c70497d17875bfcdabb 100644 (file)
@@ -19,7 +19,7 @@ var_dump(finfo_open(FILEINFO_MIME, '/foo/bar/inexistent'));
 
 ?>
 --EXPECTF--
-finfo_open(): Argument #2 ($arg) must not contain any null bytes
+finfo_open(): Argument #2 ($magic_database) must not contain any null bytes
 resource(%d) of type (file_info)
 resource(%d) of type (file_info)
 
index fea2c54041d2d0fe9ef4d35d963cc290c121f75f..b079de682d80f2a49273751c2f67afeecc8ce24f 100644 (file)
@@ -36,5 +36,5 @@ bool(false)
 
 Warning: finfo_open(): using regular magic file `%smagic' in %sfinfo_open_error.php on line %d
 resource(6) of type (file_info)
-finfo_open(): Argument #1 ($options) must be of type int, string given
-finfo::__construct(): Argument #1 ($options) must be of type int, string given
+finfo_open(): Argument #1 ($flags) must be of type int, string given
+finfo::__construct(): Argument #1 ($flags) must be of type int, string given