]> granicus.if.org Git - php/commitdiff
Remove the "auto" encoding
authorNikita Popov <nikita.ppv@gmail.com>
Wed, 17 Oct 2018 10:47:45 +0000 (12:47 +0200)
committerNikita Popov <nikita.ppv@gmail.com>
Wed, 17 Oct 2018 10:50:24 +0000 (12:50 +0200)
"auto" is only meaningful in functions which accept an encoding
*list* and support encoding detection. These functions have
explicit checks for "auto". It cannot be used as a standalone
encoding in any meaningful capacity, so I'm dropping it entirely.

ext/mbstring/libmbfl/mbfl/mbfl_encoding.c
ext/mbstring/libmbfl/mbfl/mbfl_encoding.h
ext/mbstring/tests/mb_preferred_mime_name.phpt

index 77c5daeeffbaf44e9698b3b782ec7af1ec1508da..094962090d1bcfeae7ad1bacbd7fa0ccf2fd0771 100644 (file)
 #endif
 
 
-static const char *mbfl_encoding_auto_aliases[] = {"unknown", NULL};
-
-static const mbfl_encoding mbfl_encoding_auto = {
-       mbfl_no_encoding_auto,
-       "auto",
-       NULL,
-       (const char *(*)[])&mbfl_encoding_auto_aliases,
-       NULL,
-       0,
-       NULL,
-       NULL
-};
-
 static const mbfl_encoding *mbfl_encoding_ptr_list[] = {
        &mbfl_encoding_pass,
-       &mbfl_encoding_auto,
        &mbfl_encoding_wchar,
        &mbfl_encoding_byte2be,
        &mbfl_encoding_byte2le,
index 2952977fd88e6431c1451b446ca206247f58cf47..c3a9c0affc7050a95be10d4ddd2fedf776142ac8 100644 (file)
@@ -36,7 +36,6 @@
 enum mbfl_no_encoding {
        mbfl_no_encoding_invalid = -1,
        mbfl_no_encoding_pass,
-       mbfl_no_encoding_auto,
        mbfl_no_encoding_wchar,
        mbfl_no_encoding_byte2be,
        mbfl_no_encoding_byte2le,
index 1fa73421f69160bd3cc05dbbfc1ac8d066a29f9c..df05027c3c67c783e6cd45d0340c3683de58adae 100644 (file)
@@ -38,7 +38,7 @@ echo "== INVALID PARAMETER ==\n";
 var_dump(mb_preferred_mime_name('BAD_NAME'));
 
 // No preferred name
-var_dump(mb_preferred_mime_name('auto'));
+var_dump(mb_preferred_mime_name('pass'));
 ?>
 --EXPECTF--
 Shift_JIS
@@ -55,5 +55,5 @@ UCS-4
 Warning: mb_preferred_mime_name(): Unknown encoding "BAD_NAME" in %s on line %d
 bool(false)
 
-Warning: mb_preferred_mime_name(): No MIME preferred name corresponding to "auto" in %s on line %d
+Warning: mb_preferred_mime_name(): No MIME preferred name corresponding to "pass" in %s on line %d
 bool(false)