Skip to content

Commit a85adb1

Browse files
committedFeb 22, 2023
Remove unneeded function mbfl_name2no_encoding
1 parent e934c5c commit a85adb1

File tree

3 files changed

+3
-11
lines changed

3 files changed

+3
-11
lines changed
 

‎ext/mbstring/libmbfl/mbfl/mbfl_encoding.c

-6
Original file line numberDiff line numberDiff line change
@@ -212,12 +212,6 @@ const mbfl_encoding *mbfl_no2encoding(enum mbfl_no_encoding no_encoding)
212212
return NULL;
213213
}
214214

215-
enum mbfl_no_encoding mbfl_name2no_encoding(const char *name)
216-
{
217-
const mbfl_encoding *encoding = mbfl_name2encoding(name);
218-
return encoding ? encoding->no_encoding : mbfl_no_encoding_invalid;
219-
}
220-
221215
const char *mbfl_no_encoding2name(enum mbfl_no_encoding no_encoding)
222216
{
223217
const mbfl_encoding *encoding = mbfl_no2encoding(no_encoding);

‎ext/mbstring/libmbfl/mbfl/mbfl_encoding.h

-1
Original file line numberDiff line numberDiff line change
@@ -248,7 +248,6 @@ static inline zend_string* mb_convert_buf_result(mb_convert_buf *buf, const mbfl
248248

249249
MBFLAPI extern const mbfl_encoding *mbfl_name2encoding(const char *name);
250250
MBFLAPI extern const mbfl_encoding *mbfl_no2encoding(enum mbfl_no_encoding no_encoding);
251-
MBFLAPI extern enum mbfl_no_encoding mbfl_name2no_encoding(const char *name);
252251
MBFLAPI extern const mbfl_encoding **mbfl_get_supported_encodings(void);
253252
MBFLAPI extern const char *mbfl_no_encoding2name(enum mbfl_no_encoding no_encoding);
254253
MBFLAPI extern const char *mbfl_no2preferred_mime_name(enum mbfl_no_encoding no_encoding);

‎ext/mbstring/mbstring.c

+3-4
Original file line numberDiff line numberDiff line change
@@ -1469,21 +1469,20 @@ PHP_FUNCTION(mb_substitute_character)
14691469
/* {{{ Return the preferred MIME name (charset) as a string */
14701470
PHP_FUNCTION(mb_preferred_mime_name)
14711471
{
1472-
enum mbfl_no_encoding no_encoding;
14731472
char *name = NULL;
14741473
size_t name_len;
14751474

14761475
ZEND_PARSE_PARAMETERS_START(1, 1)
14771476
Z_PARAM_STRING(name, name_len)
14781477
ZEND_PARSE_PARAMETERS_END();
14791478

1480-
no_encoding = mbfl_name2no_encoding(name);
1481-
if (no_encoding == mbfl_no_encoding_invalid) {
1479+
const mbfl_encoding *enc = mbfl_name2encoding(name);
1480+
if (enc == NULL) {
14821481
zend_argument_value_error(1, "must be a valid encoding, \"%s\" given", name);
14831482
RETURN_THROWS();
14841483
}
14851484

1486-
const char *preferred_name = mbfl_no2preferred_mime_name(no_encoding);
1485+
const char *preferred_name = mbfl_encoding_preferred_mime_name(enc);
14871486
if (preferred_name == NULL || *preferred_name == '\0') {
14881487
php_error_docref(NULL, E_WARNING, "No MIME preferred name corresponding to \"%s\"", name);
14891488
RETVAL_FALSE;

0 commit comments

Comments
 (0)
Please sign in to comment.