Fix #79462: method_exists and property_exists incoherent behavior

Both functions are closely related, so should behave the same for wrong
input types, i.e. both should throw a TypeError.

Add test to make sure internal param default eval doesn't error

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #78434

Fixed bug #78434

The DO_INIT flag, which will skip the first resume on a primed
generator, should always be set when starting to yield from a
new generator, not only when the yield from happens during priming.

Store default parameter values of internal functions in arg info

Closes GH-5353. From now on, PHP will have reflection information
about default values of parameters of internal functions.

Co-authored-by: Nikita Popov <nikita.ppv@gmail.com>

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add php_cli_server_connect() helper

Add php_cli_server_connect() helper

To encapsulate the repeated fsockopen() code.

This gives us a chance to control the timeout in one place:
Raise it to one second.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Make mysqli_poll test more deterministic

Make mysqli_poll test more deterministic

Handle errors appearing in different order.

Fix typo in GD stub

Update mb_strrpos() stub

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

Dump information about trace side exits

Merge branch 'PHP-7.4'

* PHP-7.4:
  Relax overly strict test expectation

Relax overly strict test expectation

There is no reason to expect a `1` after the PID; neither the session
ID nor the memory usage are required to contain one.  Actually, we just
want to verify here, that the process with the $child_pid is running,
and is a php.exe process.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix memory leak introduced by fixing bug #78221

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix memory leak introduced by fixing bug #78221

Fix memory leak introduced by fixing bug #78221

We have to free the retrieved text content; to keep the code readable,
we extract a helper function to check for empty nodes.  Unfortunately,
we cannot use xmlIsBlankNode(), because that also recognizes whitespace
only text content.

We also make sure to properly handle NULL returns from
xmlNodeGetContent().

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix test

Fix test

Update stack type, only if necessary

Fixed TYPE/INFO mismatch

Merge branch 'PHP-7.4'

* PHP-7.4:
  Revert "Went to fast and forgot to update tests"
  Revert "Fix Bug #79448 0 is a valid Unicode codepoint, but mb_substitute_character(0) fails"

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Revert "Went to fast and forgot to update tests"
  Revert "Fix Bug #79448 0 is a valid Unicode codepoint, but mb_substitute_character(0) fails"

Revert "Went to fast and forgot to update tests"

This reverts commit 656eac74fa6074aebc087bb73d2e4651f7dc8c9e.

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

This commit brings some substantial changes in behaviour due to the weird implementation.
This will be fixed in master due to BC concerns.

This reverts commit 1333b46d6dc0c293c1fd626803f91bc69743eb79.

cleanup

Revert "cleanup" (wrong commit)

This reverts commit 5db5f71f2831df4d32484a5638f6f6aa72e364a5.

cleanup

Remove most uses of _default_get_name()

Instead fetch the name from the respective structure. The only
place where this is still used is ReflectionClassConst,
as zend_class_const does not store the name.

Eliminate uses of _default_load_name()

Instead fetch the name from the function/class/property,
as appropriate. This makes us independent of the property,
and eliminates error conditions related to it.

Optimize internal name fetching in reflection

Directly fetch the name property, instead of construction the
properties hash table and performing a lookup in it. This is both
slow and wastes a lot of memory.

Assert that arginfo parameter name is present

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #78221: DOMNode::normalize() doesn't remove empty text nodes

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #78221: DOMNode::normalize() doesn't remove empty text nodes

Fix #78221: DOMNode::normalize() doesn't remove empty text nodes

If a text node is not followed by another text node, we remove it, if
its textContent is empty.

Free room for information about register allocation at astact stack and at trace_exit_info. Implement simple deoptimizer.

Save CPU regesters on side exit for deoptimization

Minor register allocator refactoring

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #69264: __debugInfo() ignored while extending SPL classes

Fix #69264: __debugInfo() ignored while extending SPL classes

We actually implement `::__debugInfo()` and drop the `get_debug_info()`
handlers of all relevant SPL classes.  This is cleaner and gives more
flexibility regarding overriding the functionality in descendant
classes.

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.