Don't add guard for empty()

Fixed bug #79987 (Memory leak in SplFileInfo because of missing zend_restore_error_handling())

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  7.3.23 is next

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  7.3.23 is next

7.3.23 is next

Merge branch 'PHP-7.4' into master

Prepare for 7.4.11

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Fix HTTP response status code

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix HTTP response status code

Fix HTTP response status code

JIT extension may be NULL

Fixed support for named arguments

Fixed trace type inference for FE_FETCH_R/RW

Fixed support for possible indirect variable modification (ext/standard/tests/array/bug77135.phpt failure).

JIT for ZEND_ISSET_ISEMPTY_CV

Fixed JIT for indirectly recursive traces

Promote warnings in ext/zip

Removed outdated/duplicated code

Properly patch jmp tables

Promote various OpenSSL warnings into Errors

Closes GH-5111

Add `run-tests.php --context [n]` option.

Mentioned in https://github.com/php/php-src/pull/5965#discussion_r467621123

This PR proposes 3 lines of context so the impact can be seen in tests.
Other `diff` programs show around 3 lines of context.
(This helps indicate exactly which position a test should be updated
to add a new expected line at)

Use the mapping for choosing order to display diffs

Properly include context in cases where the expected output had more lines than
the actual output, e.g.

```
--FILE--
A
A1
A
C
NEARBY
--EXPECTF--
A
B
A1
B
A
B
A
B
NEARBY
```

Closes GH-5968

Promote warnings to exceptions in ext/curl

Closes GH-5963

Merge branch 'PHP-7.4'

* PHP-7.4:
  Null terminate the sodium_crypto_kx_keypair() result

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Null terminate the sodium_crypto_kx_keypair() result

Null terminate the sodium_crypto_kx_keypair() result

Reduce cost factors in sodium password_hash tests

Drop the 4x factors to make these tests a bit less obscenely
slow.

Improve test portability

We have to ensure that the attempted connection to the MySQL server
fails, and do that by passing an unknown host instead of falling back
to localhost.

Avoid fatal error in session handler

Use variadic signature for PDOStatement::fetchAll()

The current signature is incorrect, because the $ctor_args parameter
is not required to be an array (it can at least also be null, and
isn't enforced by an exception anyway).

I'm going for the variadic signature here, because we already use
the same variadic signature in PDO::query() and
PDOStatement::setFetchMode(), all of them accepting essentially the
same arguments.

Fix types in openssl stub

These two $recipcert parameters don't use proper union types
right now. They are a bit tricky due to the $recipkey -> $recipcert
fallback.

Fix newly introduced test for Windows

The warning message is rather different there, but since that is
irrelevant for this test case, we just suppress the warning.

Add a few callable method parameter types to ext/sqlite3

Closes GH-5988

Don't assert mysql->mysql is non-null

There is an edge case in constructor behavior where we can end up
with mysql->mysql being NULL (rather than mysql itself already being
NULL). I think that ultimately that's a bug in the constructor code,
and we should probably be destroying the outer structure on
construction failure as well. However it's pretty hard to unravel
with when considering all the construction permutations.

Simplify mysqli_stmt_bind_param() implementation

By using zpp.

Use normal zpp in mysqli_result::__construct()

This method doesn't need any special handling...

Remove dummy aliases for mysqli constructors

These are only available as methods. Directly declare them as
such instead of aliasing to functions that don't actually exist.

Return empty string from SplFileInfo::getPathname()

Instead of false. This is consistent with how other methods like
SplFileInfo::getPath() behave. It's also a requirement before
SplFileInfo::__toString() calls SplFileInfo::getPathname() and
needs to return a string.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add missing initialization check to RegexIterator::getRegex()

Add missing initialization check to RegexIterator::getRegex()

Fix test after is_callable change

Fix bug #78770

Refactor the zend_is_callable implementation to check callability
at a particular frame (this is an implementation detail for now,
but could be exposed in the API if useful). Pick the first parent
user frame as the one to check.

Fix ZPP for mhash()

Closes GH-5985

Use ZPP callable check for ldap_set_rebind_proc()

Use ZPP callable check for Windows specific functions

Allow number_format to be only passed 3 arguments

Closes GH-5977

Colorize --show-diff, only on stdout

Don't add colors to the saved `.diff` file.

Related to a41cf3e1d2ed3ae38ad6115a8b65e5f14dae0b49

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79724

Fixed bug #79724

Add some missing DatePeriod initialization checks

Add a missing null check in simplexml

Fix invalid zpp calls in pcntl_get/setpriority

Fix test after PDOStatement::fetchObject() change

Use zpp C in pg_fetch_object

Use Z_PARAM_CLASS in PDOStatement::fetchObject()

Instead of implementing custom logic.

Throw Error on uninitialized XMLWriter

Throw on uninitialized SimpleXMLElement

Elevate this warning into an Error, as usual. Add a few checks
in places that were missing them.

Avoid fatal error on not found class in mysqli_fetch_object()

Instead use C zpp modifier and throw TypeError.

Check missing parameter types in stubs

[skip ci] Closes GH-5627

Fix param name in PHPDoc

Add all the missing parameter types to stubs

Closes GH-5955

Use a dedicated method for MultipleIterator::countIterators()

This method has a different signature from
SplObjectStorage::count(), so don't share implementations.

Replace fatal error in SplFileInfo with exception

Using RuntimeException here, because the same error condition
uses that in many other places in this file.

Throw Error exception in DOM_GET_OBJ

Per general convention for handling of uninitialized objects.

Switch to mime-db as source of extension => MIME map

The Apache MIME type map is not actively maintained anymore, so
this switches to jshttp/mime-db, which seems to be the de-facto
standard in this area now. This avoid the need to patch in our
own MIME types over time.

The preference algorithm is based on:
https://github.com/jshttp/mime-types/blob/47b62ac45e9b176a2af35532d0eea4968bb9eb6d/index.js#L154

Closes GH-5764.

Make strftime tests musl compatible

 * Remove usage of strftime() in favor of date() in cases where
   we are not specifically testing strftime(). We implement
   date() ourselves, and as such are insulated from implementation-
   defined behavior.
 * Add skipif for broken strftime() %Z support. We have decided
   not to work around the issue for musl using manual expansion,
   as people should not be using this function anyway, and it is
   slated for future deprecation.
 * Don't test strftime() with invalid format specifier. The
   behavior is implementation-dependent.

fix test expectation

Drop redundant arg count check

Support NO_COLOR environment variable in run-tests.php

And add a --color option for run-tests.php

See https://no-color.org/

> an informal standard is hereby proposed:
>
> All command-line software which outputs text with ANSI color added should check
> for the presence of a `NO_COLOR` environment variable that, when present
> (regardless of its value), prevents the addition of ANSI color.

Recognize gnu-libiconv location on alpine

Fix crypt_r detection under BSD system

For BSD systems, crypt.h does not exist, instead we need to
include unistd.h.

Closes GH-5971.

Small CS fix

The comments were incorrectly aligned. Fix this by dropping the
alignment and moving the comments before the declarations.

Closes GH-5975.

Disable report_zend_debug by default

We might just want to drop this completely, but at least don't
enable it by default. It already gets disabled by a number of
SAPIs, but we should make that the default state.

Rename standard array function parameters to $array

This is targeting 8.0.

`$arg` seems like a poor choice of a name,
especially if the function were to have arguments added.

In many cases, the php.net documentation already has $array for these functions.
E.g. https://www.php.net/manual/en/function.array-intersect.php

I'd assume that since named arguments was added to 8.0 near the feature freeze,
PHP's maintainers had planned to make the names consistent
and gradually use the same name for  docs and implementation.

Fix ?-> in encaps vars without braces

Closes GH-5966.

Drop php_pdo_get_exception_base() function

This API is more confusing than helpful.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #64705 errorInfo property of PDOException is null when PDO::__construct() fails

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #64705 errorInfo property of PDOException is null when PDO::__construct() fails

Fix #64705 errorInfo property of PDOException is null when PDO::__construct() fails

PDO driver constructors are throwing PdoException without setting
errorInfo, so create a new reusable function that throws exceptions
for PDO and will also set the errorInfo. Use this function in
pdo_mysql, pdo_sqlite, and pdo_pgsql.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix bug #75785 by attempt switching endianness on Maker's Note

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix bug #75785 by attempt switching endianness on Maker's Note

Fix bug #75785 by attempt switching endianness on Maker's Note

Different manufacturer models may come with a
different endianness (motorola/intel) format. In
order to avoid a big refactor and a gigantic lookup
table, this commit simply attempts to switch the
endianness and proceed when values are acceptable.

Closes GH-5849.

Fixed bug #79927

We need to unset the AT_FIRST_YIELD flag when yielding from an
array as well.

In the interest of being conservative, I'm applying this only to
PHP 8.

Fix nullsafe operator on $this

SAVE_OPLINE in NULL_HANDLER

Let's avoid crashing before the nice error message gets printed...

Fix nullsafe operator on reference

Dereference the value before checking the type. As the happy path
necessarily has to check for references, I'm not bothering to
delay the comparison.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79917

Fixed bug #79917

op_arrays can be shared on two levels: Either the op_array is
completely shared, or it is distinct but shares all members
(apart from static_variables).

The the op_array is distinct, we need to make sure to properly
initialize the MAP_PTR structures.

Autogenerate function entries for COMPersistHelper

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

Remove practically unused parameter

The `cached` out parameter of `php_com_load_typelib_via_cache()` was
meant to signal whether a particular typelib actually has been cached.
This is not really relevant, though, for the imagined purposes, and
since the parameter is no longer really used, we removed it altohether.

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Fix #48585: com_load_typelib holds reference, fails on second call

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #48585: com_load_typelib holds reference, fails on second call

Fix #48585: com_load_typelib holds reference, fails on second call

Whether the type library is cached is actually irrelevant here; what
matters is that the symbols are imported, and since these are not
cached, we have to import them for every request.  And we cannot cache
the symbols, because the import depends on the current codepage, but
the codepage is a `PHP_INI_ALL` setting.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79951

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79951

Fixed bug #79951

One branch did not release tmp_replace_entry_str.

Also reduce the scope of some variables.

Remove unnecessary function entry array

Merge branch 'PHP-7.4'

Merge branch 'PHP-7.3' into PHP-7.4

Make sure string is NUL byte terminated.

Fix signature of callback function

Cf. <https://docs.microsoft.com/en-us/windows/win32/api/imagehlp/nc-imagehlp-pimagehlp_status_routine>.