Reindent phpt files

Enable formatting of phpt files in tidy.php

Fix indentation/trailing whitespaces of 32-bit tests

Fix indentation/trailing whitespaces of tests

Merge branch 'PHP-7.4'

* PHP-7.4:
  Apply tidy formatting

Apply tidy formatting

Mostly reindent PHP scripts to spaces.

Add bundled jit libraries to exclude list

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add tidy.php to enforce formatting

Add tidy.php to enforce formatting

Many parts are disabled for the PHP-7.4 branch. We only strip
trailing whitespace in C files and reindent .php files to spaces.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Relax test expectation

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Relax test expectation

Relax test expectation

Since we're dealing with floating point numbers, precision issues may
hit us, and actually it's not necessary to check for the exact number
anyway, because it is not exact in the first place.  Therefore, we
relax the test expectations.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79212: NumberFormatter::format() may detect wrong type

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79212: NumberFormatter::format() may detect wrong type

Fix #79212: NumberFormatter::format() may detect wrong type

We have to convert to number *before* detecting the type, to cater to
internal objects implementing `cast_object`.

We also get rid of the fallback behavior of using `FORMAT_TYPE_INT32`,
because that can no longer happen; after `convert_scalar_to_number_ex`
the type is either `IS_LONG` or `IS_DOUBLE`.  We cater explicitly to
the `IS_ARRAY` case what also avoids triggering a type confusion when
`::TYPE_INT64` is passed as `$type`.

zip ext is now 1.17.1

Export zend_type_to_string() with ZEND_API

Merge branch 'PHP-7.4'

* PHP-7.4:
  add test
  NEWS
  Fixed bug #73119 Wrong return for ZipArchive::addEmptyDir Method

add test

NEWS

Fixed bug #73119 Wrong return for ZipArchive::addEmptyDir Method

Merge branch 'PHP-7.4'

* PHP-7.4:
  NEWS
  Fixed bug #73119 Wrong return for ZipArchive::addEmptyDir Method

NEWS

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add WHITESPACE_SENSITIVE run-tests section

Add WHITESPACE_SENSITIVE run-tests section

This is used to indicate that the test should not be changed by
automated formatting changes.

Fixed bug #73119 Wrong return for ZipArchive::addEmptyDir Method

Merge branch 'PHP-7.4'

* PHP-7.4:
  Disable parallelism for FPM tests

Disable parallelism for FPM tests

Let's see if this helps with spurious failures on Azure.

Fix another batch of indentation in tests

Clean up the generation of the parsers

Prefer '%define api.value.type' to '#define YYSTYPE', so that Bison
know the type.

Use '%code requires' to declare what is needed to define the api.value.type
(that code is output in the generated header before the generated
definition of YYSTYPE).

Prefer '%define api.prefix' inside the grammar file to '-p' outside,
as anyway the functions defined in the file actually use this prefix.

Prefer `%param` to both `%parse-param` and `%lex-param`.

Closes GH-5138

Bison: enable all the warnings and fix them

First, fix 5547d361208d90e12d53bb62bb2ffbbff9b93ca0: the definition of
YFLAGS was not passed into the Makefile: AC_SUBST does not suffice, we
need PHP_SUBST_OLD.  While at it, allow to pass variable and value at
the same time.

Then pass -Wall to bison, rather than only -Wempty-rules.

Use %precedence where associativity is useless.

Remove useless %precedence.
GH-5138

Added opcache extension to inis

Extensions are not loaded by default (commented out)

Closes GH-5136

Fix var_dump test

Fix indentation and whitespaces in tests

In preparation for GH-5074

inline by hand to avoid uninitialized variable warning

When compiling with "-Wall -Werror" gcc emits two errors:

../src/pcre2_jit_neon_inc.h:211:8: error: ‘cmp1b’ is used uninitialized in this function [-Werror=uninitialized]
  211 | data = fast_forward_char_pair_compare(compare1_type, data, cmp1a, cmp1b);
      |        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../src/pcre2_jit_neon_inc.h:212:9: error: ‘cmp2b’ is used uninitialized in this function [-Werror=uninitialized]
  212 | data2 = fast_forward_char_pair_compare(compare2_type, data2, cmp2a, cmp2b);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

The compiler emits the error message before inlining.
Because the warning is based on an intra-procedural data
flow analysis, the compiler does not see that cmp1b and
cmp2b are not used when they are not initialized.

Here is the code of that function, cmp2 is only used
when ctype is compare_match2 or compare_match1i,
and not when ctype is compare_match1:

static inline vect_t fast_forward_char_pair_compare(compare_type ctype, vect_t dst, vect_t cmp1, vect_t cmp2)
{
if (ctype == compare_match2)
 {
 vect_t tmp = dst;
 dst = VCEQQ(dst, cmp1);
 tmp = VCEQQ(tmp, cmp2);
 dst = VORRQ(dst, tmp);
 return dst;
 }

if (ctype == compare_match1i)
  dst = VORRQ(dst, cmp2);
dst = VCEQQ(dst, cmp1);
return dst;
}

The patch inlines by hand the case of compare_match1 such that the
code avoids referring to cmp1b and cmp2b.

Tested on aarch64-linux with `make check`.

We don't need "safe" destruction anymore

These EG(current_execute_data) = EX(prev_execute_data) assignments are useless now

Export zend_dump_op() and add ZEND_DUMP_NUMERIC_OPLINES flag to print oplines as "dddd" instead of "Ld+"

Fix JIT as well

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix bug #76047

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix bug #76047

Fix bug #76047

Unlink the current stack frame before freeing CVs or extra args.
This means it will no longer show up in back traces that are
generated during CV destruction.

We already did this prior to destructing the object/closure,
presumably for the same reason.

Use "%empty" in the parsers, instead of comments

The annotation %empty is properly enforced: warnings when it's
missing, and errors when it's inappropriate.  Support for %empty was
introduced in Bison 3.0.

Pass -Wempty-rule to Bison.

Closes GH-5134

Zip: version is now 1.17.0
change for Windows (which have libzip 1.4 by default)
update NEWS

add ZipArchive::registerProgressCallback and ZipArchive::registerCancelCallback methods

Fixed JIT part for bug #79094

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79094 (Crashing when running recursion function)

Fixed bug #79094 (Crashing when running recursion function)

Add NEWS entry for new ReflectionProperty methods.

Promote mysqli warnings to exceptions

Closes GH-5058

Revert "Make BG(syslog_device) per request"

This reverts commit b0d7b126a29a1972229ac91d6d28f5331912eddb.

This change wasn't quite right: I noticed only now that the
RSHUTDOWN function is #ifdef PHP_WIN32 and I'm not fully convinced
that ifdef can be removed: syslog might also be used by error
logging in FPM for example, in which case we probably shouldn't
be closing the log here.

Generally it's unclear how the openlog() functionality exposed
by PHP is supposed to interact with openlog() uses by SAPIs.

For now I'll just revert this change and let it leak across
requests.

Merge branch 'PHP-7.4'

* PHP-7.4:
  fix cross compilation failure due to size_t typecast in define

fix cross compilation failure due to size_t typecast in define

The following commit introduces a cross-compilation failure:

   93c728b77cfb47f5cfdd1863f8982ea59d344205
  "Try to control ZEND_MM_ALIGNED_SIZE type"

br-arm-full/build/php-7.4.2/Zend/zend_alloc.h:30:38:
error: missing binary operator before token "8"
                                              ^
br-arm-full/build/php-7.4.2/ext/opcache/ZendAccelerator.c:1380:7:
note: in expansion of macro ‘ZEND_MM_ALIGNMENT’

Closes GH-5128.

Initialize SplFixedArray elements to NULL instead of UNDEF

The SplFixedArray API treats all elements as NULL, even if they
have not been explicitly initialized. Rather than initializing
to UNDEF an treating that specially in various circumstances,
directly initialize elements to NULL.

This also fixes an assertion failure in the attached test case.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79193

Fixed bug #79193

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix live range calculation for FE_FETCH

Fix live range calculation for FE_FETCH

Op2 is def here, not a use, so treat it accordingly.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix DatePeriod property handling with indirect modification

Fix DatePeriod property handling with indirect modification

We do need to implement get_property_ptr_ptr to make arrays work
correctly.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #70078: XSL callbacks with nodes as parameter leak memory

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #70078: XSL callbacks with nodes as parameter leak memory

Fix #70078: XSL callbacks with nodes as parameter leak memory

The fix for bug #49634 solved a double-free by copying the node with
`xmlDocCopyNodeList()`, but the copied node is later freed by calling
`xmlFreeNode()` instead of `xmlFreeNodeList()`, thus leaking memory.
However, there is no need to treat the node as node list, i.e. to copy
also the node's siblings; just creating a recursive copy of the node
with `xmlDocCopyNode()` is sufficient, while that also avoids the leak.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix leak in DateTimeImmutable::modify()

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix leak in DateTimeImmutable::modify()

Fix leak in DateTimeImmutable::modify()

Update test for warning -> fatal error change

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix copying of functions in variance obligations

Fix copying of functions in variance obligations

Only copy sizeof(zend_internal_function) for internal functions.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add SKIPIF to test requiring mbregex

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Add SKIPIF to test requiring mbregex

Add SKIPIF to test requiring mbregex

Fix leak in DatePeriod construction with invalid format

Same issue as I fixed in DateInterval construction before.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Reset trampoline on executor startup
  Fix UAF in is_callable() and allocated trampoline

Reset trampoline on executor startup

Make sure the trampoline is usable, even if we had an unclean
shutdown on the last request.

Fix UAF in is_callable() and allocated trampoline

By nulling out the function_handler, so it will not get used
below. Reuse the existing helper for this purpose.

Fix #74063: NumberFormatter fails after retrieval from session

While it would be desireable to actually support unserialization of
NumberFormatter instances, at least we should not allow serialization
for now.

We also remove some doubtful tests, which have been added[1] claiming
that they would crash the intl extension, but apparently no fix has
been applied, and the test cases have not been marked as XFAIL.

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

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix shift ub in mbstring
  Restore digit check in mb_decode_numericentity()

Fix shift ub in mbstring

Ideally "c" would be an unsigned integer...

Restore digit check in mb_decode_numericentity()

I replaced it with a multiplication overflow check in
18599f9c52959b2e8cbfac57e278644499a3547d. However, we need both,
because the code for restoring the number can't handle numbers
with many leading zeros right now and I don't feel like teaching it.

Enable ZipArchive::setMtime(Name|Index) on Windows

These are enabled on non Windows systems as of zip 1.16.0 with libzip
>= 1.0.0.  Since Windows builds use at least libzip 1.4.0, we also
enable these methods there.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix mb_ord() crash if internal encoding not supported

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix mb_ord() crash if internal encoding not supported

Fix mb_ord() crash if internal encoding not supported

enc_name can be NULL here. Take the name from the mbfl_encoding
instead.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Better overflow check for entity decoding

Better overflow check for entity decoding

Check for multiplication overflow rather than number of digits.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Reset MBREX(search_re) in RSHUTDOWN

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Reset MBREX(search_re) in RSHUTDOWN

Reset MBREX(search_re) in RSHUTDOWN

This is going to cause a segfault if reused in the next request.
To illustrate the issue, run these two scripts in sequence with
the built-in server:

// script1.php
mb_ereg_search_init('foobar');
mb_ereg_search('foo');

// script2.php
var_dump(mb_ereg_search_init("foobar"));
var_dump(mb_ereg_search_pos());

zip extension version is now 1.16.1

Merge branch 'PHP-7.4'

* PHP-7.4:
  zip: more constants

zip: more constants

Merge branch 'PHP-7.4'

* PHP-7.4:
  Backport 7d2ef3d2e540885dec26d91dad061bff1621ad07 into 7.4

Backport 7d2ef3d2e540885dec26d91dad061bff1621ad07 into 7.4

As the data structures are public, the fix for 64-bit consists
on replacing the blanket memcpy with individual assignments.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix use of mb_ereg_search_getregs() after invalid pattern

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix use of mb_ereg_search_getregs() after invalid pattern

Fix use of mb_ereg_search_getregs() after invalid pattern

This segfaulted because we assumed that if there are matches,
there must be a regular expression as well.

Fix datatype

This has been introduced by 84b0d0fabaaf21ee056984a33b573121296af942.
Besides it causes runtime issues on POWER5 (and presumably later), the
implementation would expect this array to consist on 32-bit integers.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix length inconsistency in mb_convert_encoding

Fix length inconsistency in mb_convert_encoding

Don't mix strlen() and ZSTR_LEN(). If the encoding contains a
NULL byte, this will overflow the buffer.

NULL bytes will still make this behave oddly because the consuming
code will cut off the string there, but let's address that in master...