Fix IntlGregorianCalendar constructor signature

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

Side exit on overflow

Throw correct exception from ArrayObject sort methods

Let normal zpp throw ArgumentCountErrors.

Add missing param to DatePeriod::__construct()

The signature is very overloaded, but takes at most four params,
not three.

JMP optimization

Eliminate useless exception checks

Eliminate some reference-counting

Merge branch 'PHP-7.4'

* PHP-7.4:
  Revert "Drop freetype from i386"

Revert "Drop freetype from i386"

This reverts commit 8641d76f62273f3a73449fd720c93c63992f3b5a.

This appears to work again now...

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix azure i386 build

Fix azure i386 build

Purge libsqlite3-0 to make sure we get the i386 library lateron.

Fix WeakReference::create() stub

cleanup

Fix #79805: sapi_windows_vt100_support throws TypeError

It does not make sense to throw a `TypeError` when the stream can't be
analyzed.  If `sapi_windows_vt100_support()` is used as getter, we just
return `false` in that case; if the function is used as setter, we
additionally trigger a warning.

We also fix the test cases for this function, which have been broken
before.  Note that these tests are still whitespace sensitive.

Fix #79846: 8c67c166996 broke simple regexp

We backport the respective upstream fix[1].

[1] <https://vcs.pcre.org/pcre2?view=revision&revision=1267>

cleanup

Merge branch 'PHP-7.4'

* PHP-7.4:
  Enable further tests on Windows

Enable further tests on Windows

Check type guard on result of FETCH_OBJ_R/IS instructions

Merge branch 'PHP-7.4'

* PHP-7.4:
  #79657 was cherry-picked in 7.4.7

#79657 was cherry-picked in 7.4.7

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix readlink related memory leak

Fix readlink related memory leak

Fixed tracing JIT for ASSIGN to typed reference

Ignore inheritance rules on private methods

Closes GH-5401

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix tests regarding negative fileinode()

Fix tests regarding negative fileinode()

The results of `fileinode()` may be negative due to wrap-around
behavior (at least on Windows as of PHP 7.4.0).

Check type guard on result of FETCH_DIM_R/IS instructions

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix bug #78008: dns_check_record() always return true on Alpine

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix bug #78008: dns_check_record() always return true on Alpine

Fix bug #78008: dns_check_record() always return true on Alpine

- free handle before return result
- cleaned up remaining usage of MAXPACKET
- update dns_get_mx() to use the same approach

Closes GH-5854.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79862

Fixed bug #79862

While normally a private property in the active scope would take
priority, we should not use this if it has the wrong "staticness".

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79820

Fixed bug #79820

Similar to what is done for ReflectionType itself, copy the
type name stored inside ReflectionProperty. Also make sure the
type field is always initialized for dynamic properties.

This is a non-issue in PHP 8, because we store a pointer to the
property_info there, rather than a copy.

Remove type argument from zend_get_zval_ptr

As pointed out by Levi, this is unused, and we already changed
the signature of this function for PHP 8.

Retrofit description of recent PR changes

Reduce scope of preload compiler_options

Only set preloading compiler_options while executing the preload
file, not when performing linking afterwards. Otherwise options
like IGNORE_INTERNAL_CLASSES will hide classes from inheritance
verification.

Fix preloading of union typed property inherited from internal class

Tricky edge case: We inherit a property from an internal class,
in which case no property duplication takes place. We should not
try to persist the property info in that case. This didn't really
matter previously, but now that the property has some non-interned
owned data (the type), we need to make sure we don't try to free
that.

Check for name vs ce in a few more places

The type lists may contain CEs, we should not assume they only
contain names.

Tracing JIT for SWITCH instructions

Better support for 64-bit .aword constants

Fix preloading property type resolution

This was causing many failures under --preload...

Merge branch 'PHP-7.4'

* PHP-7.4:

Merge remote-tracking branch 'upstream/PHP-7.4' into PHP-7.4

* upstream/PHP-7.4:
  Fix test for x86 Windows

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79849

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79849

Fixed bug #79849

Closes GH-5853.

Fixed bug #79852

Use different scratch register

r1 clashes with FCARG1a on Windows, so use a different scratch
register. FCARG2a should be free here. Hopefully fixes the
AppVeyor build.

Support socketpairs in proc_open()

Closes GH-5777.

Remove out-dated comment

ZTS support has been added around ~7.2.

Fix typos

Put debug function behind ifdef

Add user upgrading note for no_separation change

I added this to UPGRADING.INTERNALS, but it should also be
mentioned in UPGRADING, as it affects user-visible behavior.

[ci skip]

Add upgrading note for crypt change

[ci skip]

Restore warning

Use consistent types

uint32_t type for argument count
size_t for length of char*
zend_bool for a zval bool arg

Closes GH-5845

Improved reference-counting inference

Add upgrading note

[ci skip]

define COMPILER and ARCHITECTURE from environment

Improve output of tokens in Parse Errors

Currently, unexpected tokens in the parser are shown as the text
found, plus the internal token name, including the notorious
"unexpected '::' (T_PAAMAYIM_NEKUDOTAYIM)".

This commit replaces that with a more user-friendly format, with
two main types of token:

* Tokens which always represent the same text are shown like
  'unexpected token "::"' and 'expected "::"'
* Tokens which have variable text are given a user-friendly
  name, and show like 'unexpected identifier "foo"', and
  'expected identifer'.

A few tokens have special cases:

* unexpected token """ -> unexpected double-quote mark
* unexpected quoted string "'foo'" -> unexpected single-quoted
  string "foo"
* unexpected quoted string ""foo"" -> unexpected double-quoted
  string "foo"
* unexpected illegal character "_" -> unexpected character 0xNN
  (where _ is almost certainly a control character, and NN is the
   hexadecimal value of the byte)

The \ token has a special case in the implementation just to stop
bison making a mess of escaping it and it coming out as \\

JIT for array merging

Fix compilation errors when EXIF_DEBUG is defined

Fix warnings and error when compiling with EXIF_DEBUG defined.

Closes GH-5838.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix test for x86 Windows

Fix test for x86 Windows

`st_dev` deliberately overflows on such systems, cf.
<http://svn.php.net/viewvc?view=revision&revision=350100>.

Add another doctrine workaround

This has been fixed a few months ago, but apparently Doctrine
doesn't do releases.

Check for undef var in verify return jit

Fixed bug #79828

Fixed bug #79841

Fix PDO_Firebird numeric to string conversion on Windows

Commit aaa1f90[1] dropped support for the `snprinf()` `I` modifier, so
we use the standard `ll` modifier which is suitable for 32bit and 64bit
Windows.  We also replace the deprecated `I64` suffix[2] with the `LL`
suffix.

This fixes ext/pdo_firebird/tests/bug_64037.phpt.

[1] <http://git.php.net/?p=php-src.git;a=commit;h=aaa1f90e3f90c24098fa55a7b868fdca0b89ee25>
[2] <https://docs.microsoft.com/en-us/cpp/cpp/numeric-boolean-and-pointer-literals-cpp?view=vs-2019#integer-literals>

Fix PDO_Firebird tests which assume ERRMODE_SILENT

These have apparently been missed when PR 5388[1] had been merged.

[1] <https://github.com/php/php-src/pull/5388>

Unbork tests for PDO_Firebird

PDO_Firebird raises a warning regarding the unknown SQL syntax, which
we have to silence to avoid run-tests.php to mark the test as borked.

Fix free of uninitialized memory in MATCH_ERROR

As suggested by Tyson Andre:
https://github.com/php/php-src/pull/5371#issuecomment-657081464

Also fix line number of unhandled match error

Closes GH-5841.

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

A complement to ae6f45ad45e072260d93f828d81fee9548123044

Closes GH-5843.

Addd upgrading notes about OCI8

Removed the "oci_internal_debug" function and its alias. Also deprecated the "ocifetchinto" function.

Closes GH-5810

(ffi) fix ifdefs for non-windows systems without long double

Closes GH-5840

Export php_gd_libgdimageptr_from_zval_p()

Some extension may need to retrieve the `gdImagePtr` from an `GdImage`
object; thus, we export the respective function.  To not being forced
to include gd.h in php_gd.h, we use the opaque `struct gdImageStruct *`
as return type.

We also rename php_gd2.dll to php_gd.dll, since there's not really much
point in giving the DLL a version number, since there is no php_gd.dll
for years (if there ever has been).  Renaming, on the other hand,
matches the name on other systems (gd.so), and allows to actually use
`ADD_EXTENSION_DEP()`.

[skip ci] Consistent match test closing tags

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed BC break of php_debug_zval_dump

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed BC break of php_debug_zval_dump

Fixed BC break of php_debug_zval_dump

It introduced by fixing bug #79830

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79830 introduced by fixing bug #79821

# Conflicts:
# ext/standard/var.c

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79830 introduced by fixing bug #79821

Fixed bug #79830 introduced by fixing bug #79821

This also fixes memory error in debug_zval_dump and var_export.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79821

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79821

Fixed bug #79821

HashTable was reallocated (zend_hash_packed_grow) during php_var_dump, so we should call GC_ADDREF to make SEPARATE_ARRAY work.

Closes GH-5837.

Don't generate JSON scanner and parser during configure

Retain the original formatting character in test

Review the usage of apostrophes in error messages

Closes GH-5590

Fix test cases

These tests are not supposed to check the result of `fclose(false)`.

Use dedicated function for illegal offset type error

Add missing addref for MATCH_ERROR operand

Promote Notice to Warning in libmagic

The current behaviour makes little to no sense, having a notice which starts with 'Warning:' should just be a warning

Closes GH-5824

Fixed bug #79818

Only destroy the variable directly before reassigning it. The
value could be read in the meantime.

Add stubs for SAPIs

Closes GH-5295.

Fix warning in embed sapi

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79817