Normalize mb_ereg() return value

mb_ereg()/mb_eregi() currently have an inconsistent return value
based on whether the $matches parameter is passed or not:

> Returns the byte le

Normalize mb_ereg() return value

mb_ereg()/mb_eregi() currently have an inconsistent return value
based on whether the $matches parameter is passed or not:

> Returns the byte length of the matched string if a match for
> pattern was found in string, or FALSE if no matches were found
> or an error occurred.
>
> If the optional parameter regs was not passed or the length of
> the matched string is 0, this function returns 1.

Coupling this behavior to the $matches parameter doesn't make sense
-- we know the match length either way, there is no technical
reason to distinguish them. However, returning the match length
is not particularly useful either, especially due to the need to
convert 0-length into 1-length to satisfy "truthy" checks. We
could always return 1, which would kind of match the behavior of
preg_match() -- however, preg_match() actually returns the number
of matches, which is 0 or 1 for preg_match(), while false signals
an error. However, mb_ereg() returns false both for no match and
for an error. This would result in an odd 1|false return value.

The patch canonicalizes mb_ereg() to always return a boolean,
where true indicates a match and false indicates no match or error.
This also matches the behavior of the mb_ereg_match() and
mb_ereg_search() functions.

This fixes the default value integrity violation in PHP 8.

Closes GH-6331.

show more ...

Update mbstring parameter names

Closes GH-6207.

Standardize mbstring and string on using 'string' as a parameter name.

Closes GH-6171.

Promote a few remaining errors in ext/standard

Closes GH-6110

Make mb_send_mail() consistent with mail()

The $additional_headers parameter shouldn't accept null.

Move custom type checks to ZPP

Closes GH-6034

Add a few missing parameter types in stubs

Related to GH-5627

Fix UNKNOWN default values in ext/mbstring and ext/gd

Closes GH-5598

Fix mb_ereg_search* arg and func info

Fix [-Wundef] warning in MBString extension

Refactor mb_substitute_character()

Using the new Fast ZPP API for string|int|null

This also fixes Bug #79448 which was too disruptive to fix in PHP 7.x

Convert if blocks to assertions and adapt stubs accordingly

Update mb_strrpos() stub

We no longer accept the encoding as 3rd param, so we can make this
a proper int argument.

Generate function entries for another batch of extensions

Closes GH-5352

Promote invalid case mode to ValueError in mb_case_converter

Add assertions to check the return value is not NULL as this indicates a bug.
Add identical assertion to mb_strtoupper and mb

Promote invalid case mode to ValueError in mb_case_converter

Add assertions to check the return value is not NULL as this indicates a bug.
Add identical assertion to mb_strtoupper and mb_strtolower.
This means these functions can't return false anymore, ammend stubs accordingly.

show more ...

mb_scrub() can't return false anymore

Also drop the intermediary function which was only used here

Refactor mb_substr_count()

Promote empty needle warning to ValueError
Convert if branch into an assertion as if mbfl_substr_count fails this now implies a bug
Thus mb_substr_count()

Refactor mb_substr_count()

Promote empty needle warning to ValueError
Convert if branch into an assertion as if mbfl_substr_count fails this now implies a bug
Thus mb_substr_count() can only return int now, fix stubs accordingly

show more ...

Convert if branch to assertion in mb_strlen

This operation should never fail, therefore it's converted to an assertion.
Thus this mb_strlen() can now only return int, fix stubs according

Convert if branch to assertion in mb_strlen

This operation should never fail, therefore it's converted to an assertion.
Thus this mb_strlen() can now only return int, fix stubs accordingly

show more ...

Promote mb_str_split warning to ValueError

Also add a TODO about documenting this funcion on PHP.net
Convert some checks to assertions as if they don't hold something went wrong during m

Promote mb_str_split warning to ValueError

Also add a TODO about documenting this funcion on PHP.net
Convert some checks to assertions as if they don't hold something went wrong during memory allocation
Due to these changes this function cannot return false anymore, fix stubs accordingly

show more ...

mb_detect_order(): Use proper array|string argument

mb_check_encoding(): Make var a proper array|string arg

mb_detect_encoding(): Use proper array|string parameter

Needed to add support for nullabiltiy in some places.

mb_convert_variables(): Make $from an array|string argument

mb_convert_encoding(): Make $input a proper array|string arg

mb_convert_encoding(): Make $from_encodings a proper array|string arg

Switching to FastZPP, as we don't support this in normal zpp.