Use ZPP callable check for oci_register_taf_callback()

Some cleanup in OCI8 extension for PHP 8

Disable tracing JIT on AppVeyor

This is a temporary measure until the current issues[1] are resolved.

[1] <https://github.com/php/php-src/commit/0e1e991a89528b571a0271a3f24f9d3c2c36e23a#commitcomment-40805659>

Fix deoptimization info and corresponding memory leaks

Fixed bug #79864 (JIT segfault in Symfony OptionsResolver)

Support optional comma in closure use list

RFC: https://wiki.php.net/rfc/trailing_comma_in_closure_use_list
Discussion: https://externals.io/message/110715

The release manager has agreed to allow merging of RFCs that have near-unanimous
votes. If an RFC ends up not achieving the required 2/3 majority at the time the
announced voting period closes, this implementation commit will be reverted
in time for the feature freeze.

Closes GH-5793

Improve argument types and names in ext/ctype

Closes GH-5878

Give zend_pass_function an arginfo

Now that the ZEND_ACC_VARIADIC flag is set, we should also make
sure there is a variadic arg that can be looked up.

Tweak zend_test arginfo/zpp

Fix incorrect initialization in date_sunrise()

Switch to tracing JIT by default

Fix arg/func info

 * mysqli_get_server_info() cannot return null. The underlying API
   is infallible.
 * mysqli_select_db() func info is redundant.
 * mb_detect_order() can only return array|true, not array|false.

Also make the func_info.phpt test that is supposed to catch these
kinds of issues actually work.

Cleanup argument handling of Zend functions and methods

Closes GH-5847

add ZipArchive::CM_XZ constant (in upcoming libzip 1.8.0)

Fix type recording and side exit information for FE_FETCH_* instructions

Run arginfo / zpp mismatch tests for methods as well

As a side-effect, this also catches issues where classes are not
prepared for use with newInstanceWithoutConstructor.

Check for missing arginfo arguments

Internal functions error when too many arguments are passed. Make
this part of the verification we do in debug builds. This will
help avoid cases where an argument is missing in the stubs,
as recently encountered in 6d96f0f.

Avoid recording of uninitialized variable

Check for redundant func info

Separate __call and __soapCall implementations

This is overly pedantic, but allows us to enable more arginfo
consistency checks.

Add stubs for PDO PGSql extension methods

Add stubs for PDO SQLite extension methods

Putting these under a dummy PDO_SQLite_Ext class.

Disable JIT for PHPDBG

Fix UConvert::getErrorMessage() leak on zpp failure

Move SOAP_SERVER_BEGIN_CODE() after zpp

This is still very dubious, because there are lots of other "returns"
between the BEGIN and END -- won't that end up not restoring the
original state?

Make ReflectionGenerator final

This class is not safe against malicious extension / instantiation.

Stricter verification of func info against arg info

Make sure they're actually the same up to cases where func info
allows more accurate expressions. There are some awkward edge cases
around true/false/null limitations in union types.

Adjust Doctrine workaround in community job

[ci skip]

Switch to tracing JIT by default

Use different temporary register

Next is PHP 8.0.0beta1

Remove some redundant func info

This is fully covered by arginfo.

The array_merge_recursive() RC information was also wrong,
it should be the same as array_merge().

Update NEWS for PHP 8.0.0alpha3

Fixed support for 64-bit constants

Fix mb_ereg_search* arg and func info

Fix readline test after callback changes

This one requires libedit and was probably missed for that reason.

Remove null from highlight_* return types

Also fix show_source() discrepancy in func_info.

Merge branch 'PHP-7.4'

Prepare for 7.4.10

Remove bool return type from assert_options

Not seeing any way this function can return bool.

Fix iptcembed func info

This function can return true with $spool >= 2.

Call zpp_none in PharFileInfo::__destruct()

Using __destruct() with internal classes is dubious, but not so
simple to avoid here because the code extends SPL classes.

Merge branch 'PHP-7.4'

* PHP-7.4:
  7.3 is now 7.3.22-dev

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  7.3 is now 7.3.22-dev

7.3 is now 7.3.22-dev

Fixed pg_select() function info

Use ZPP callable check in readline extension

Use ZPP callable for PDO Sqlite

Fixed tracing JIT exception handling

Explode _php_image_output_ctx() into multiple functions

Closes GH-5844

Fixed VM interrupt handling

Keep CPU regesters used by deoptimizer

Fixed tracing JIT for VM without global register variables

Preserve original ce_flags when registering class

Bug that regularly sneaks in: ZEND_ACC_FINAL is set before calling
zend_register_internal_class() and promptly gets ignored. Remove
this footgun by preserving flags from the original CE.

Fetch pdo stmt after zpp

Update method signature in test

Use zpp for PDO fetch mode

Also changing the function signatures to accept variadic args
for the fetch params. If we're already breaking Doctrine anyway,
we may as well do it properly.

Fix bug #79857: Add upgrading note for exit dtor change

[ci skip]

Fix bug #79868

This simply restores the code from PHP 7.4 which I incorrectly
"simplified" in master.

Add common code for magic method assignment

This was repeated three times.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #63527: DCOM does not work with Username, Password parameter

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #63527: DCOM does not work with Username, Password parameter

Fix #63527: DCOM does not work with Username, Password parameter

We must not mix multibyte and wide character strings in the
`COAUTHIDENTITY` structure.  Using wide character strings throughout
would have the advantage that the remote connection can be established
regardless of the code page of the server, but that would more likely
break BC, so we just drop the wide character string conversion of the
username.

Fixed incorrect deoptimization info

Directly assign magic methods

Instead of going through intermediary variables.

Remove unused lc_class_name variable

This is probably a leftover from "old style constructor" support.

Report magic method names as written

Report the name the way the user has written it, the same way we
always do.

Unify magic method visibility check

This was missing entirely for the internal function case.

Remove old code (BP_VAR_RW warning)

Fix tests I missed before

Unify magic method return type checks

Unify static/non-static check for magic methods

And promote it to be fatal.

Add missing arg num check for __set_state

Clean up zend_check_magic_method_implementation

Let everything go through a common function, which fixes some
consistency issues.

Merge branch 'oci8-3.0'

* oci8-3.0:
  Rename OCI8 classes to use standard names so arg stub parsing works
  Bump OCI8 version to 3.0 so a PHP 8 PECL release can be made

Rename OCI8 classes to use standard names so arg stub parsing works

Bump OCI8 version to 3.0 so a PHP 8 PECL release can be made

Revert "Switch to tracing JIT by default"

This reverts commit 10b09578a89a444d9df45e75d5ce8c0826bb2efe.

This seems to cause many test failures:

Test === operator : different types [tests/lang/operators/operator_identical_basic.phpt]
Test <=> operator : different types [tests/lang/operators/operator_spaceship_basic.phpt]
comparing different variables for equality [Zend/tests/compare_001_64bit.phpt]
comparing different variables for identity [Zend/tests/compare_002_64bit.phpt]
Rebinding of ::getClosure()s [Zend/tests/closure_061.phpt]
comparing different variables (greater than) [Zend/tests/compare_003_64bit.phpt]
comparing different variables (less than) [Zend/tests/compare_004_64bit.phpt]
comparing different variables (greater or equal than) [Zend/tests/compare_005_64bit.phpt]
comparing different variables (smaller or equal than) [Zend/tests/compare_006_64bit.phpt]
Invalid numeric string E_WARNINGs and E_NOTICEs, combined assignment operations [Zend/tests/numeric_string_errors_assign.phpt]
Bug #51819 (Case discrepancy in timezone names cause Uncaught exception and fatal error) [ext/date/tests/bug51819.phpt]
Test all filters returned by filter_list() [ext/filter/tests/033.phpt]
filter_var() and FILTER_VALIDATE_MAC [ext/filter/tests/055.phpt]
Test strrpos() function : usage variations - unexpected inputs for 'needle' argument [ext/standard/tests/strings/strrpos_variation10.phpt]
Test strtr() function : usage variations - empty string & null for 'str' argument [ext/standard/tests/strings/strtr_variation4.phpt]
Test strtr() function : usage variations - unexpected inputs for 'from' argument [ext/standard/tests/strings/strtr_variation6.phpt]
Test strstr() function [ext/standard/tests/strings/strstr.phpt]

Switch to tracing JIT by default

Skip upload_2G.phpt if disk_free_space() <= 2GB

This test failed when the free disk space is close to 2.15GB.
I see the file size in the .out file as 0.
PHP has to save the full file contents to disk (the path is in `$_FILES`)

Related to GH-5283

Closes GH-5873

Use proper methods for php_user_filter implementation

These are dummy methods, but should at least check zpp for
arginfo consistency.

Move SessionHandler sanity checks after zpp

Only set DIT type on successful initialization

Remove spurious return true from DOMXPath::registerPhpFunctions()

Only one of three identical cases returned true, and the function
is documented to return void. Remove the outlier.

Use string|array type in DOMXPath::registerPhpFunctions()

Add missing zpp_none call in Exception::__wakeup

Convert SPL fatal error to Error exception

Drop useless zend_accel_script_persistable() function

Drop unused param in zend_parse_arg_class_name_or_obj()

Add error notification callbacks to UPGRADING.INTERNALS

Parse parameters in zend_test_void_return()

Fix some pgsql arginfos

And make sure too few/many arguments throws, to satisfy the
function signature.

Some zpp cleanup in pgsql

Remove some zpp quiet uses in pgsql

These functions don't do anything special, just use normal zpp.

Add missing image_type_to_extension() argument

Fixed bug #79867

In line with usual rules, give untyped properties a null default
value. Otherwise constructor promotion would give you a property
declaration that cannot be achieved through any other means.

Introduce error notification callbacks that are run independant of zend_error_cb

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79839

Fixed bug #79839

Add reference type sources in array_walk.

Fix IntlGregorianCalendar constructor signature

Give these conversative UNKNOWN defaults and no types, as the
overload is something of a mess.