Skip to content

Commit a06c20a

Browse files
committed
Remove useless constant MBFL_ENCTYPE_MBCS
This flag indicated that an encoding was 'multi-byte'; it can use a variable number of bytes to encode each character. As it turns out, we don't actually need to check this flag anywhere, so it's better to remove it.
1 parent 6cbeb64 commit a06c20a

30 files changed

+42
-49
lines changed

ext/mbstring/libmbfl/filters/mbfilter_big5.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ const mbfl_encoding mbfl_encoding_big5 = {
5959
"BIG5",
6060
mbfl_encoding_big5_aliases,
6161
mblen_table_big5,
62-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
62+
MBFL_ENCTYPE_GL_UNSAFE,
6363
&vtbl_big5_wchar,
6464
&vtbl_wchar_big5
6565
};
@@ -70,7 +70,7 @@ const mbfl_encoding mbfl_encoding_cp950 = {
7070
"BIG5",
7171
NULL,
7272
mblen_table_big5,
73-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
73+
MBFL_ENCTYPE_GL_UNSAFE,
7474
&vtbl_cp950_wchar,
7575
&vtbl_wchar_cp950
7676
};

ext/mbstring/libmbfl/filters/mbfilter_cp5022x.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ const mbfl_encoding mbfl_encoding_cp50220 = {
5858
"ISO-2022-JP",
5959
cp50220_aliases,
6060
NULL,
61-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
61+
MBFL_ENCTYPE_GL_UNSAFE,
6262
&vtbl_cp50220_wchar,
6363
&vtbl_wchar_cp50220
6464
};
@@ -69,7 +69,7 @@ const mbfl_encoding mbfl_encoding_cp50221 = {
6969
"ISO-2022-JP",
7070
NULL,
7171
NULL,
72-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
72+
MBFL_ENCTYPE_GL_UNSAFE,
7373
&vtbl_cp50221_wchar,
7474
&vtbl_wchar_cp50221
7575
};
@@ -80,7 +80,7 @@ const mbfl_encoding mbfl_encoding_cp50222 = {
8080
"ISO-2022-JP",
8181
NULL,
8282
NULL,
83-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
83+
MBFL_ENCTYPE_GL_UNSAFE,
8484
&vtbl_cp50222_wchar,
8585
&vtbl_wchar_cp50222
8686
};

ext/mbstring/libmbfl/filters/mbfilter_cp51932.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ const mbfl_encoding mbfl_encoding_cp51932 = {
6363
"CP51932",
6464
mbfl_encoding_cp51932_aliases,
6565
mblen_table_eucjp,
66-
MBFL_ENCTYPE_MBCS,
66+
0,
6767
&vtbl_cp51932_wchar,
6868
&vtbl_wchar_cp51932
6969
};

ext/mbstring/libmbfl/filters/mbfilter_cp932.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ const mbfl_encoding mbfl_encoding_cp932 = {
6262
"Shift_JIS",
6363
mbfl_encoding_cp932_aliases,
6464
mblen_table_sjis,
65-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
65+
MBFL_ENCTYPE_GL_UNSAFE,
6666
&vtbl_cp932_wchar,
6767
&vtbl_wchar_cp932
6868
};

ext/mbstring/libmbfl/filters/mbfilter_cp936.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ const mbfl_encoding mbfl_encoding_cp936 = {
5959
"CP936",
6060
mbfl_encoding_cp936_aliases,
6161
mblen_table_cp936,
62-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
62+
MBFL_ENCTYPE_GL_UNSAFE,
6363
&vtbl_cp936_wchar,
6464
&vtbl_wchar_cp936
6565
};

ext/mbstring/libmbfl/filters/mbfilter_euc_cn.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ const mbfl_encoding mbfl_encoding_euc_cn = {
5959
"CN-GB",
6060
mbfl_encoding_euc_cn_aliases,
6161
mblen_table_euccn,
62-
MBFL_ENCTYPE_MBCS,
62+
0,
6363
&vtbl_euccn_wchar,
6464
&vtbl_wchar_euccn
6565
};

ext/mbstring/libmbfl/filters/mbfilter_euc_jp.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ const mbfl_encoding mbfl_encoding_euc_jp = {
6262
"EUC-JP",
6363
mbfl_encoding_euc_jp_aliases,
6464
mblen_table_eucjp,
65-
MBFL_ENCTYPE_MBCS,
65+
0,
6666
&vtbl_eucjp_wchar,
6767
&vtbl_wchar_eucjp
6868
};

ext/mbstring/libmbfl/filters/mbfilter_euc_jp_2004.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ const mbfl_encoding mbfl_encoding_eucjp2004 = {
4141
"EUC-JP",
4242
mbfl_encoding_eucjp2004_aliases,
4343
mblen_table_eucjp,
44-
MBFL_ENCTYPE_MBCS,
44+
0,
4545
&vtbl_eucjp2004_wchar,
4646
&vtbl_wchar_eucjp2004
4747
};

ext/mbstring/libmbfl/filters/mbfilter_euc_jp_win.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ const mbfl_encoding mbfl_encoding_eucjp_win = {
6161
"EUC-JP",
6262
mbfl_encoding_eucjp_win_aliases,
6363
mblen_table_eucjp,
64-
MBFL_ENCTYPE_MBCS,
64+
0,
6565
&vtbl_eucjpwin_wchar,
6666
&vtbl_wchar_eucjpwin
6767
};

ext/mbstring/libmbfl/filters/mbfilter_euc_kr.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ const mbfl_encoding mbfl_encoding_euc_kr = {
5858
"EUC-KR",
5959
mbfl_encoding_euc_kr_aliases,
6060
mblen_table_euckr,
61-
MBFL_ENCTYPE_MBCS,
61+
0,
6262
&vtbl_euckr_wchar,
6363
&vtbl_wchar_euckr
6464
};

ext/mbstring/libmbfl/filters/mbfilter_euc_tw.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ const mbfl_encoding mbfl_encoding_euc_tw = {
6060
"EUC-TW",
6161
mbfl_encoding_euc_tw_aliases,
6262
mblen_table_euctw,
63-
MBFL_ENCTYPE_MBCS,
63+
0,
6464
&vtbl_euctw_wchar,
6565
&vtbl_wchar_euctw
6666
};

ext/mbstring/libmbfl/filters/mbfilter_gb18030.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ const mbfl_encoding mbfl_encoding_gb18030 = {
4141
"GB18030",
4242
mbfl_encoding_gb18030_aliases,
4343
NULL,
44-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
44+
MBFL_ENCTYPE_GL_UNSAFE,
4545
&vtbl_gb18030_wchar,
4646
&vtbl_wchar_gb18030
4747
};

ext/mbstring/libmbfl/filters/mbfilter_hz.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ const mbfl_encoding mbfl_encoding_hz = {
3838
"HZ-GB-2312",
3939
NULL,
4040
NULL,
41-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
41+
MBFL_ENCTYPE_GL_UNSAFE,
4242
&vtbl_hz_wchar,
4343
&vtbl_wchar_hz
4444
};

ext/mbstring/libmbfl/filters/mbfilter_iso2022_jp_ms.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ const mbfl_encoding mbfl_encoding_2022jpms = {
4242
"ISO-2022-JP",
4343
mbfl_encoding_2022jpms_aliases,
4444
NULL,
45-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
45+
MBFL_ENCTYPE_GL_UNSAFE,
4646
&vtbl_2022jpms_wchar,
4747
&vtbl_wchar_2022jpms
4848
};

ext/mbstring/libmbfl/filters/mbfilter_iso2022_kr.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ const mbfl_encoding mbfl_encoding_2022kr = {
3737
"ISO-2022-KR",
3838
NULL,
3939
NULL,
40-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
40+
MBFL_ENCTYPE_GL_UNSAFE,
4141
&vtbl_2022kr_wchar,
4242
&vtbl_wchar_2022kr
4343
};

ext/mbstring/libmbfl/filters/mbfilter_iso2022jp_2004.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ const mbfl_encoding mbfl_encoding_2022jp_2004 = {
4141
"ISO-2022-JP-2004",
4242
NULL,
4343
NULL,
44-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
44+
MBFL_ENCTYPE_GL_UNSAFE,
4545
&vtbl_2022jp_2004_wchar,
4646
&vtbl_wchar_2022jp_2004
4747
};

ext/mbstring/libmbfl/filters/mbfilter_iso2022jp_mobile.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ const mbfl_encoding mbfl_encoding_2022jp_kddi = {
4545
"ISO-2022-JP",
4646
mbfl_encoding_2022jp_kddi_aliases,
4747
NULL,
48-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
48+
MBFL_ENCTYPE_GL_UNSAFE,
4949
&vtbl_2022jp_kddi_wchar,
5050
&vtbl_wchar_2022jp_kddi
5151
};

ext/mbstring/libmbfl/filters/mbfilter_jis.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ const mbfl_encoding mbfl_encoding_jis = {
4141
"ISO-2022-JP",
4242
NULL,
4343
NULL,
44-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
44+
MBFL_ENCTYPE_GL_UNSAFE,
4545
&vtbl_jis_wchar,
4646
&vtbl_wchar_jis
4747
};
@@ -52,7 +52,7 @@ const mbfl_encoding mbfl_encoding_2022jp = {
5252
"ISO-2022-JP",
5353
NULL,
5454
NULL,
55-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
55+
MBFL_ENCTYPE_GL_UNSAFE,
5656
&vtbl_2022jp_wchar,
5757
&vtbl_wchar_2022jp
5858
};

ext/mbstring/libmbfl/filters/mbfilter_sjis.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ const mbfl_encoding mbfl_encoding_sjis = {
6565
"Shift_JIS",
6666
mbfl_encoding_sjis_aliases,
6767
mblen_table_sjis,
68-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
68+
MBFL_ENCTYPE_GL_UNSAFE,
6969
&vtbl_sjis_wchar,
7070
&vtbl_wchar_sjis
7171
};

ext/mbstring/libmbfl/filters/mbfilter_sjis_2004.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ const mbfl_encoding mbfl_encoding_sjis2004 = {
4646
"Shift_JIS",
4747
mbfl_encoding_sjis2004_aliases,
4848
mblen_table_sjis,
49-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
49+
MBFL_ENCTYPE_GL_UNSAFE,
5050
&vtbl_sjis2004_wchar,
5151
&vtbl_wchar_sjis2004
5252
};

ext/mbstring/libmbfl/filters/mbfilter_sjis_mac.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ const mbfl_encoding mbfl_encoding_sjis_mac = {
4848
"Shift_JIS",
4949
mbfl_encoding_sjis_mac_aliases,
5050
mblen_table_sjis,
51-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
51+
MBFL_ENCTYPE_GL_UNSAFE,
5252
&vtbl_sjis_mac_wchar,
5353
&vtbl_wchar_sjis_mac
5454
};

ext/mbstring/libmbfl/filters/mbfilter_sjis_mobile.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ const mbfl_encoding mbfl_encoding_sjis_docomo = {
5050
"Shift_JIS",
5151
mbfl_encoding_sjis_docomo_aliases,
5252
mblen_table_sjis,
53-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
53+
MBFL_ENCTYPE_GL_UNSAFE,
5454
&vtbl_sjis_docomo_wchar,
5555
&vtbl_wchar_sjis_docomo
5656
};
@@ -61,7 +61,7 @@ const mbfl_encoding mbfl_encoding_sjis_kddi = {
6161
"Shift_JIS",
6262
mbfl_encoding_sjis_kddi_aliases,
6363
mblen_table_sjis,
64-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
64+
MBFL_ENCTYPE_GL_UNSAFE,
6565
&vtbl_sjis_kddi_wchar,
6666
&vtbl_wchar_sjis_kddi
6767
};
@@ -72,7 +72,7 @@ const mbfl_encoding mbfl_encoding_sjis_sb = {
7272
"Shift_JIS",
7373
mbfl_encoding_sjis_sb_aliases,
7474
mblen_table_sjis,
75-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
75+
MBFL_ENCTYPE_GL_UNSAFE,
7676
&vtbl_sjis_sb_wchar,
7777
&vtbl_wchar_sjis_sb
7878
};

ext/mbstring/libmbfl/filters/mbfilter_sjis_open.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ const mbfl_encoding mbfl_encoding_sjis_open = {
6060
"Shift_JIS",
6161
mbfl_encoding_sjis_open_aliases,
6262
mblen_table_sjis,
63-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
63+
MBFL_ENCTYPE_GL_UNSAFE,
6464
&vtbl_sjis_open_wchar,
6565
&vtbl_wchar_sjis_open
6666
};

ext/mbstring/libmbfl/filters/mbfilter_uhc.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ const mbfl_encoding mbfl_encoding_uhc = {
5959
"UHC",
6060
mbfl_encoding_uhc_aliases,
6161
mblen_table_uhc,
62-
MBFL_ENCTYPE_MBCS,
62+
0,
6363
&vtbl_uhc_wchar,
6464
&vtbl_wchar_uhc
6565
};

ext/mbstring/libmbfl/filters/mbfilter_utf7.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ const mbfl_encoding mbfl_encoding_utf7 = {
5151
"UTF-7",
5252
mbfl_encoding_utf7_aliases,
5353
NULL,
54-
MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE,
54+
MBFL_ENCTYPE_GL_UNSAFE,
5555
&vtbl_utf7_wchar,
5656
&vtbl_wchar_utf7
5757
};

ext/mbstring/libmbfl/filters/mbfilter_utf7imap.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ const mbfl_encoding mbfl_encoding_utf7imap = {
8686
NULL,
8787
mbfl_encoding_utf7imap_aliases,
8888
NULL,
89-
MBFL_ENCTYPE_MBCS,
89+
0,
9090
&vtbl_utf7imap_wchar,
9191
&vtbl_wchar_utf7imap
9292
};

ext/mbstring/libmbfl/filters/mbfilter_utf8.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ const mbfl_encoding mbfl_encoding_utf8 = {
5757
"UTF-8",
5858
mbfl_encoding_utf8_aliases,
5959
mblen_table_utf8,
60-
MBFL_ENCTYPE_MBCS,
60+
0,
6161
&vtbl_utf8_wchar,
6262
&vtbl_wchar_utf8
6363
};

ext/mbstring/libmbfl/filters/mbfilter_utf8_mobile.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ const mbfl_encoding mbfl_encoding_utf8_docomo = {
4747
"UTF-8",
4848
mbfl_encoding_utf8_docomo_aliases,
4949
mblen_table_utf8,
50-
MBFL_ENCTYPE_MBCS,
50+
0,
5151
&vtbl_utf8_docomo_wchar,
5252
&vtbl_wchar_utf8_docomo
5353
};
@@ -58,7 +58,7 @@ const mbfl_encoding mbfl_encoding_utf8_kddi_a = {
5858
"UTF-8",
5959
mbfl_encoding_utf8_kddi_a_aliases,
6060
mblen_table_utf8,
61-
MBFL_ENCTYPE_MBCS,
61+
0,
6262
&vtbl_utf8_kddi_a_wchar,
6363
&vtbl_wchar_utf8_kddi_a
6464
};
@@ -69,7 +69,7 @@ const mbfl_encoding mbfl_encoding_utf8_kddi_b = {
6969
"UTF-8",
7070
mbfl_encoding_utf8_kddi_b_aliases,
7171
mblen_table_utf8,
72-
MBFL_ENCTYPE_MBCS,
72+
0,
7373
&vtbl_utf8_kddi_b_wchar,
7474
&vtbl_wchar_utf8_kddi_b
7575
};
@@ -80,7 +80,7 @@ const mbfl_encoding mbfl_encoding_utf8_sb = {
8080
"UTF-8",
8181
mbfl_encoding_utf8_sb_aliases,
8282
mblen_table_utf8,
83-
MBFL_ENCTYPE_MBCS,
83+
0,
8484
&vtbl_utf8_sb_wchar,
8585
&vtbl_wchar_utf8_sb
8686
};

ext/mbstring/libmbfl/mbfl/mbfl_consts.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,6 @@
3232
#define MBFL_CONSTS_H
3333

3434
#define MBFL_ENCTYPE_SBCS 0x00000001 /* single-byte encoding */
35-
#define MBFL_ENCTYPE_MBCS 0x00000002 /* multi-byte encoding */
3635
#define MBFL_ENCTYPE_WCS2 0x00000010 /* 2 bytes/char */
3736
#define MBFL_ENCTYPE_MWC2 0x00000040 /* 2+ bytes/char */
3837
#define MBFL_ENCTYPE_WCS4 0x00000100 /* 4 bytes/char */

ext/mbstring/mbstring.c

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -403,13 +403,7 @@ static const char *php_mb_zend_encoding_name_getter(const zend_encoding *encodin
403403
static bool php_mb_zend_encoding_lexer_compatibility_checker(const zend_encoding *_encoding)
404404
{
405405
const mbfl_encoding *encoding = (const mbfl_encoding*)_encoding;
406-
if (encoding->flag & MBFL_ENCTYPE_SBCS) {
407-
return 1;
408-
}
409-
if ((encoding->flag & (MBFL_ENCTYPE_MBCS | MBFL_ENCTYPE_GL_UNSAFE)) == MBFL_ENCTYPE_MBCS) {
410-
return 1;
411-
}
412-
return 0;
406+
return !(encoding->flag & MBFL_ENCTYPE_GL_UNSAFE);
413407
}
414408

415409
static const zend_encoding *php_mb_zend_encoding_detector(const unsigned char *arg_string, size_t arg_length, const zend_encoding **list, size_t list_size)
@@ -4210,10 +4204,10 @@ static int php_mb_encoding_translation(void)
42104204
/* {{{ MBSTRING_API size_t php_mb_mbchar_bytes_ex() */
42114205
MBSTRING_API size_t php_mb_mbchar_bytes_ex(const char *s, const mbfl_encoding *enc)
42124206
{
4213-
if (enc != NULL) {
4214-
if (enc->flag & MBFL_ENCTYPE_MBCS) {
4215-
if (enc->mblen_table != NULL) {
4216-
if (s != NULL) return enc->mblen_table[*(unsigned char *)s];
4207+
if (enc) {
4208+
if (enc->mblen_table) {
4209+
if (s) {
4210+
return enc->mblen_table[*(unsigned char *)s];
42174211
}
42184212
} else if (enc->flag & MBFL_ENCTYPE_WCS2) {
42194213
return 2;

0 commit comments

Comments
 (0)