Inline a ZEND_NUM_ARGS() variable

Clean up constructor handling in com_dotnet

We substitute the construction magic with standard constructors, move
the ZPP checks to the beginning of the ctors, and also let the function
entries be generated from the stubs.

Improve some ValueError messages

Closes GH-5340

Generate function entries for another batch of extensions

Closes GH-5352

Add missing stub for xmlrpc_get_type()

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.

mb_scrub() can't return false anymore

Also drop the intermediary function which was only used here

Promote some warnings to ValueError in mbstring

Promoted warnings are:
 * Empty encoding lists
 * Unknown language
 * Start and Width out of bound

Convert some if blocks to assertions

Generate functions entries from stubs for another set of extensions

Closes GH-5351

Move variable declaration closer to its usage

Improve gen_stub.php

Closes GH-5350
Add support for generating deprecated function entries, as well as forward declaration
of function aliases.

Generate function entries from stubs for a couple of extensions

Closes GH-5347

Annotate function aliases in stubs

Merge branch 'PHP-7.4'

Merge branch 'PHP-7.3' into PHP-7.4

Went to fast and forgot to update tests

However due to the really lax conversion to integer all strings pass as 0

Merge branch 'PHP-7.4'

Merge branch 'PHP-7.3' into PHP-7.4

Fix Bug #79448 0 is a valid Unicode codepoint, but mb_substitute_character(0) fails

Fix stub for zend_terminate_string()

Verify that all stubs have a return type

Stubs: Store information per-class

We'll need this if we want to generate method entries.

Remove now unnecessary PHP_FUNCTION() declarations

Generate function entries from stubs

If @generate-function-entries is specified in the stub file,
also generate function entries for the extension.

Currently limited to free functions only.

Do not include the same stub multiple times

Closes GH-5322

Fix #68690: Hypothetical off-by-one condition

We fix this, even though `filter->cache == jisx0213_u2_tbl_len` can
never be true here.

Promote unknown encoding throws in encoding array/string list

For the string list we emit still emit a warning by comparing arg_num to 0

Closes GH-5337

add some output when generating arginfo

Move gen_stub.php to build directory and install it so phpize can take care of it, and thus extension can use it as it is already in Makefile

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix incorrect free for last_message

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix incorrect free for last_message

Fix incorrect free for last_message

In commit a7305eb539596e175bd6c3ae9a20953358c5d677 the last_message
field of the connection object was changed to be always non-persistent.
But there is a place on change_user path that still treats it
depending on conn->persistent flag. This will cause PHP crash after
com_change_user success when there is last_message set

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

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

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

Merge mb_strstr() variants under a common implementation

This reduces heavy ducplicate code

Improve code flow

These nested if-else blocks are unnecessary since we return early.

Use ZEND_NUM_ARGS() macro instead of custom variable

Also why on earth would someone do this?

Promote empty string warning to ValueError in mb_ord()

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79434

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79434

Fixed bug #79434

Merge branch 'PHP-7.4'

* PHP-7.4:
  Skip fpm tests not designed to be run as root
  Allow fpm tests to be run with long socket path

Skip fpm tests not designed to be run as root

When running as root with TEST_FPM_RUN_AS_ROOT=1

Allow fpm tests to be run with long socket path

Socket path is restricted to ~100 bytes, so we can use
the system temp dir if the path ends up too long.

news / UPGRADING for zip 1.19.0

- add ZipArchive::EM_UNKNOWN constant - add ZipArchive::EM_TRAD_PKWARE constant - cleanup hack for libzip 1.3.1 (have only exist for a few days) - add ZipArchive::isCompressionMethodSupported() method (libzip 1.7.0) - add ZipArchive::isEncryptionMethodSupported() method (libzip 1.7.0) - bump version to 1.19.0-dev

Fix #47983: mixed LF and CRLF line endings in mail()

Email headers are supposed to be separated with CRLF.  Period.

Prevent imap mail tests from borking instead of skipping

As of commit e49593a[1], run-tests.php is rather picky regarding the
output of SKIPIF sections, so we have to suppress warnings for failing
imap_open().

[1] <http://git.php.net/?p=php-src.git;a=commit;h=e49593a7c3159a5673ce74b4e5133f1264dc313f>

use a better setter for zip_error_t

Remove <default> prefix from reflection dump

This really doesn't add anything, and only makes for confusing
terminology. Only marking properties as dynamic is sufficient.

Display property default value in reflection dumps

Show property type in reflection export

Avoid control flow warning

Merge branch 'PHP-7.4'

* PHP-7.4:
  Enable error diagnostic for check_default_conf_path.phpt

Enable error diagnostic for check_default_conf_path.phpt

Otherwise we have no clue why the test failed, if the regex didn't
match.

Remove unnecessary check in runtime_compile_time_binary_operands.phpt

Now that operator errors are detected more accurately, we no longer
have any cases where we throw a compile-time error instead of a
run-time exception, so we can drop this check now.

Make division by zero error check more accurate

For division (rather than modulus) we should check the double
value, otherwise the result might be zero after integer truncation,
but not zero as a floating point value.

Unify checks for binary operator errors for ct eval

Move everything into one function and share it with opcache.
This fixes some discrepancies.

Don't convert binop operand types in opcache

This may produce different behavior if operator overloading is
involved, and may change the error message.

If there's strong interest, this could be done in the DFA pass
with available type information. It does not look particularly
practically useful to me though.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix literal compaction collision between string and double

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix literal compaction collision between string and double

Fix literal compaction collision between string and double

For the sake of simplicity I'm using a separate hashtable, rather
than trying to do hash perturabation on the double strings.

Improve "unsupported operands" error

By mentioning the operand types. We can do that now, as the
original operand types now remain available.

Closes GH-5330.

Remove op_func from TRY_BINARY_OP1 macro

And move the operator overloading helpers into zend_operators.c,
there's no reason for them to be in the header.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add NEWS entryc [ci skip]

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Add NEWS entryc [ci skip]

Add NEWS entryc [ci skip]

Merge branch 'PHP-7.4'

Merge branch 'PHP-7.3' into PHP-7.4

Fix bug 79441

Promote unsupported encoding warnings to ValueError

Use php_mb_get_encoding instead of mbfl_name2encoding to get encoding

This reduces the number of places where the error message template is used.
Also promote the mb_check_encoding() warning to ValueError and add a test to cover the behaviour.

Move encoding fetching out of php_mb_convert_encoding()

Remove unnecesary check in php_mb_check_encoding()

The caller makes sure that this is not NULL.

Refactor operator implementations

Instead of looping, use straight-line code with the following
layout:

1. Try to apply the base operation on the dereferenced operands.
2. Try overloaded object operations.
3. Try to convert operands to number, else error out.
4. Apply the base operation on the converted operands.

This makes the code easier to reason about and fixes some edge-case
bugs:

1. We should only try invoking operator overloading once prior to
   type conversion. Previously it was invoked both before and after
   type conversion.
2. We should not modify any values if an exception is thrown.
   Previously we sometimes modified the LHS of a compound assignment
   operator.
3. If conversion of the first operand fails, we no longer try to
   convert the second operand. I think the previous behavior here
   was fine as well, but this still seems a more typical.

This will also make some followup changes I have in mind simpler.

Use explicit helper

Encoding argument is always MBREX(current_mbctype)

Add additional operator exception handling tests

These show that we may lose the old value of the LHS of a compound
assignment operator in some cases.

Remove unused argument form convert_object_to_type()

Display nullability in type error messages for internal functions

Closes GH-5327

Clean up silent/noisy flag in convert_to_number

Convert some unknown encoding warnings to ValueErrors in ext/mbstring

Promotes only the warnings where the encoding comes only from a string.
Functions which accept an array of encodings will be fixed at a later stage.

Closes GH-5317

Promote some warnings to Errors in Zend basic functions

Closes GH-5325

Merge branch 'PHP-7.4'

The 7.4 branch is now for 7.4.6

Make cast_object handler required

Avoid subtle differences in behavior depending on whether the
handler is absent or returns FAILURE.

If you previously set cast_object to NULL, create a handler that
always returns FAILURE instead.

Improve undefined variable error messages

Closes GH-5312

Clarify SimpleXML comparison logic

Add a ZEND_UNCOMPARABLE value

To explicitly indicate that objects are uncomparable. For now
this has no functional difference from the usual 1 return value,
but makes intent clearer.

Report object cast failures internally

Make cast_object return FAILURE for casts to int/float, rather than
throwing a notice and returning SUCCESS. Instead move the emission
of the notice to the code invoking cast_object. This will allow us
to customize the behavior per call-site.

This change is written to be NFC, and the code in
zend_std_compare_objects() should illustrate the current behavior
doesn't make a lot of sense.

Make sure php_get_internal_encoding() returns non-empty

Even if default_charset is set to "", we should still return
"UTF-8" as the default value here. Setting default_charset to ""
suppresses the header emission, but shouldn't change anything
about our encoding defaults.

Make sure mbstring.internal_encoding deprecation is always thrown

It was not thrown if the setting was specified via -d at least.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Next is 7.3.18

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Next is 7.3.18

Next is 7.3.18

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79413: session_create_id() fails for active sessions

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79413: session_create_id() fails for active sessions

Fix #79413: session_create_id() fails for active sessions

The comment on `PS_VALIDATE_SID_FUNC(files)` is very clear that the
function is supposed to return `SUCCESS` if the session already exists.
So to detect a collision, we have to check for `SUCCESS`, not
`FAILURE`.

We also fix the wrong condition in session_regenerate_id() as well.