Deprecate required param after optional

As an exception, we allow "Type $foo = null" to occur before a
required parameter, because this pattern was used as a replacement
for nullable types in PHP versions older than 7.1.

Closes GH-5067.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove extra tab in NEWS [ci skip]

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Remove extra tab in NEWS [ci skip]

Remove extra tab in NEWS [ci skip]

Merge branch 'PHP-7.4'

* PHP-7.4:
  bump version to 7.2.29

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  bump version to 7.2.29

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  bump version to 7.2.29

bump version to 7.2.29

Add return types to FFI methods

The FFI class is final, so we can use proper return types.

Add proper return types to final phar methods

As these are final, we can add return types to them.

Remove mysqli reflection tests

Fix zpp in mysqli_warning::next()

This only exists as a method, so using zend_parse_method_parameters
doesn't make sense. Also make sure that zpp is always called, not
only conditionally.

[ci skip] I still help to maintain these extensions

or at least planning to.

Make quoting of cmd execution functions consistent

While the `$command` passed to `proc_open()` had to be wrapped in
double-quotes manually, that was implicitly done for all other
program execution functions.  This could easily introduce bugs and
even security issues when switching from one to another program
execution function.

Furthermore we ensure that the additional quotes are always
unwrapped regardless of what is passed as `$command` by passing
the `/s` flag to cmd.exe.  As it was, `shell_exec('path with
spaces/program.exe')` did execute program.exe, but adding an
argument (`shell_exec('path with spaces/program.exe -h)`) failed
to execute program.exe, because cmd.exe stripped the additional
quotes.

While these changes obviously can cause BC breaks, we feel that in
the long run the benefits of having consistent behavior for all
program execution functions outweighs the drawbacks of potentially
breaking some code now.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79038: PDOStatement::nextRowset() leaks column values

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79038: PDOStatement::nextRowset() leaks column values

Fix #79038: PDOStatement::nextRowset() leaks column values

Firstly, we must not rely on `stmt->column_count` when freeing the
driver specific column values, but rather store the column count in
the driver data.  Since the column count is a `short`, 16 bit are
sufficient, so we can store it in reserved bits of `pdo_odbc_stmt`.

Furthermore, we must not allocate new column value storage when the
statement is not executed, but rather when the column value storage has
not been allocated.

Finally, we have to introduce a driver specific `cursor_closer` to
avoid that `::closeCursor()` calls `odbc_stmt_next_rowset()` which then
frees the column value storage, because it may be still needed for
bound columns.

Rename php_dom_create_interator() to php_dom_create_iterator()

Thus fixing an obvious typo.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix NEWS [ci skip]

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix NEWS [ci skip]

Fix NEWS [ci skip]

Merge branch 'PHP-7.4'

* PHP-7.4:
  Update NEWS [ci skip]

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Update NEWS [ci skip]

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Update NEWS [ci skip]

Update NEWS [ci skip]

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix # 79171: heap-buffer-overflow in phar_extract_file
  Fix bug #79082 - Files added to tar with Phar::buildFromIterator have all-access permissions
  Fix bug #79221 - Null Pointer Dereference in PHP Session Upload Progress
  Mark bug76348.phpt as online test
  Fix bug #79082 - Files added to tar with Phar::buildFromIterator have all-access permissions
  Fix bug #79221 - Null Pointer Dereference in PHP Session Upload Progress

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix # 79171: heap-buffer-overflow in phar_extract_file
  Fix bug #79082 - Files added to tar with Phar::buildFromIterator have all-access permissions
  Fix bug #79221 - Null Pointer Dereference in PHP Session Upload Progress
  Mark bug76348.phpt as online test
  Fix bug #79082 - Files added to tar with Phar::buildFromIterator have all-access permissions
  Fix bug #79221 - Null Pointer Dereference in PHP Session Upload Progress

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Mark bug76348.phpt as online test
  Fix bug #79082 - Files added to tar with Phar::buildFromIterator have all-access permissions
  Fix bug #79221 - Null Pointer Dereference in PHP Session Upload Progress

Merge branch 'PHP-7.4'

* PHP-7.4:
  Create a new console for each test worker on Windows

Create a new console for each test worker on Windows

The primary motivation to have each test worker running its own console
is to allow the windows_mb_path tests to run in parallel.  A nice side
effect is that this also prevents changing the code page of the
tester's console window (which can even cause its font to be changed).

To be able to do so, we introduce the `create_new_console` option for
`proc_open()`, which might occasionally be useful for other purposes
than testing.

Make type error messages more consistent

Closes GH-5092

Improve generated names for anonymous classes

In order of preference, the generated name will be:

    new class extends ParentClass {};
    // -> ParentClass@anonymous
    new class implements FirstInterface, SecondInterface {};
    // -> FirstInterface@anonymous
    new class {};
    // -> class@anonymous

This is intended to display a more useful class name in error messages
and stack traces, and thus make debugging easier.

Closes GH-5153.

Add static return type

RFC: https://wiki.php.net/rfc/static_return_type

The "static" type is represented as MAY_BE_STATIC, rather than
a class type like "self" and "parent", as it has special
resolution semantics, and cannot be cached in the runtime cache.

Closes GH-5062.

Use strlen for INI_* macros

strlen() is resolved at compile time by optimizing compilers.

Prevent mistakes and allow to use INI_* macros with char * variables
which can have sizeof() == 1.

Closes GH-5174.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Disable instantiation of zero size FFI\CData objects
  Fix # 79171: heap-buffer-overflow in phar_extract_file
  Fix bug #79082 - Files added to tar with Phar::buildFromIterator have all-access permissions
  Fix bug #79221 - Null Pointer Dereference in PHP Session Upload Progress

Disable instantiation of zero size FFI\CData objects

Fixed bug #79255 (PHP cannot be compiled with enable JIT)

Revert "Fixed bug #79255 (PHP cannot be compiled with enable JIT)"

This reverts commit de6984edd5ce7fa2b2f365b5ae770507ef42022d.

Reverting to unbreak 32-bit CI.

Fix # 79171: heap-buffer-overflow in phar_extract_file

We must not access memory outside of the allocated buffer.

Fix bug #79082 - Files added to tar with Phar::buildFromIterator have all-access permissions

Fix bug #79221 - Null Pointer Dereference in PHP Session Upload Progress

Fix # 79171: heap-buffer-overflow in phar_extract_file

We must not access memory outside of the allocated buffer.

Fix bug #79082 - Files added to tar with Phar::buildFromIterator have all-access permissions

Fix bug #79221 - Null Pointer Dereference in PHP Session Upload Progress

Fix # 79171: heap-buffer-overflow in phar_extract_file

We must not access memory outside of the allocated buffer.

Fix bug #79082 - Files added to tar with Phar::buildFromIterator have all-access permissions

Fix bug #79221 - Null Pointer Dereference in PHP Session Upload Progress

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79271: DOMDocumentType::$childNodes is NULL

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79271: DOMDocumentType::$childNodes is NULL

Fix #79271: DOMDocumentType::$childNodes is NULL

Dom level 2 core, DOM level 3 core and the DOM living standard agree
that `childNodes` always return a `NodeList`, and never `null`.

Revert "Replace @param annotations with type declarations"

This reverts commit c31029f335ca1b453af799805c43c37e959ad555.

Mark bug76348.phpt as online test

Fix bug #79082 - Files added to tar with Phar::buildFromIterator have all-access permissions

Fixed bug #79255 (PHP cannot be compiled with enable JIT)

[skip ci] Skip 2 tokenizer tests if tokenizer isn't loaded

`./configure --disable-tokenizer` can disable tokenizer

Closes GH-5184

Replace @param annotations with type declarations

Constrain number parameter of numfmt_format to int|float

This is inline with similar changes to the math functions.  Especially,
array to number conversion makes no sense here, and is likely to hide
a programming error.

To make that feasible, we introduce the `n` specifier for classic ZPP
so we can stick with `zend_parse_method_parameters()`.

We also remove a test case, which has been degenerated to a ZPP test.

[skip ci] Fix typos in UPGRADING

Closes GH-5183

Fix arginfo

These parameters accept int|float, since they are parsed with
`Z_PARAM_NUMBER`.

Fix bug #79221 - Null Pointer Dereference in PHP Session Upload Progress

Update bundled config.guess and config.sub

Clarify that token_get_all() never returns false

It can only fail in TOKEN_PARSE mode, in which case it will throw.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix typo in recent bugfix

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix typo in recent bugfix

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Fix typo in recent bugfix

Fix typo in recent bugfix

Add stubs for zend_test

Add arginfo for HashContext::__construct()

Arginfo was already generated, just not referenced.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix Azure MacOS build

Fix Azure MacOS build

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #77569: Write Acess Violation in DomImplementation

Fix JIT trait type errors

We need to load EX->func here rather than use a hardcoded op_array,
as it may be copied with adjusted scope for traits.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #77569: Write Acess Violation in DomImplementation

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Fix #77569: Write Acess Violation in DomImplementation

Fix #77569: Write Acess Violation in DomImplementation

We must not assume that the zval IS_STRING.

Add const modifier for name strings

Closes GH-5173.

Improved the check

makefile.global is also used while building shared extension

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove value from comment in php.ini files

Remove value from comment in php.ini files

Closes GH-5164.

Reset required_num_args for disabled functions

Otherwise we may get arginfo/zpp mismatch errors.

var_dump(): Don't skip recursion detection on first level

This is confusing. The current output doesn't make it clear that
we're in fact recursing to the top-level structure.

Closes GH-5171.

Fixed build (Only PHP above 7.1 could run the gen_stub.php)

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79257

Fixed bug #79257

Replace an existing entry for a given name only if we have a match.

Removed unused macro

Reuse SEPARATE_ARRAY() macro

Add UPGRADING notes

Not listing the details here, I think the RFC is a better reference
for the precise technical details.

[ci skip]

Allow arbitrary expressions in new/instanceof using () syntax

Treat magic constants like normal constants

Make class constants fully dereferencable

Unify array and object dereferencability

Split into fully_dereferencable and array_dereferencable

Split class_constant from constant production

Make encapsed strings fully dereferencable

This allows operations that were previously allowed on constant
strings on interpolated strings as well.

Add support for $obj::class

This allows $obj::class, which gives the same result as get_class($obj).
Anything other than an object results in TypeError.

RFC: https://wiki.php.net/rfc/class_name_literal_on_object

Closes GH-5065.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79254: getenv() w/o arguments not showing changes

Remove CURLOPT_PASSWDDATA block CURLOPT_PASSWDDATA was removed from curl as of 7.10.8

Closes GH-5166.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79254: getenv() w/o arguments not showing changes

Fix #79254: getenv() w/o arguments not showing changes

To be able to see changes done only with `SetEnvironmentVariable()`, we
have to use `GetEnvironmentStrings()` instead of `environ`, because the
latter sees only changes done with `putenv()`.

For best backward compatibility we're using `GetEnvironmentStringsA()`;
switching to the wide string version likely makes sense for master,
though.

Revert "Fix [-Wmissing-field-initializers] compiler warning in ext/PDO"

Seems like Travis doesn't like this and now throws a
[-Werror=maybe-uninitialized] instead.

This reverts commit 051b01d465661e16f65bcfd58fcf1d7e03b26f35.