Merge branch 'PHP-8.0'

* PHP-8.0:
  Remove embedded property from mysqli_driver

Remove embedded property from mysqli_driver

All other leftovers of this feature have been dropped in PHP 8,
so we should remove the property as well.

Closes GH-6407.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed bug #80334

Fixed bug #80334

If assert() was called with named args, add description as named
arg as well.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed missaligned access

Fixed missaligned access

Merge branch 'PHP-8.0'

* PHP-8.0:
  Skip preload test under asan

Skip preload test under asan

Just like the other preload tests with startup failures, this may
cause leaks.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Rename PhpToken::getAll() to PhpToken::tokenize()

Rename PhpToken::getAll() to PhpToken::tokenize()

See https://externals.io/message/112189.
Fixes bug #80328.

Merge branch 'PHP-8.0'

Merge branch 'PHP-7.4' into PHP-8.0

Update version in 7.4 branch

Merge branch 'PHP-8.0'

* PHP-8.0:
  Split tests for compatibility with ICU 68.1

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Split tests for compatibility with ICU 68.1

Split tests for compatibility with ICU 68.1

Stabalize IMAP parallel testing, again

Drive by clean-up
Use less the default mailbox again

Merge branch 'PHP-8.0'

* PHP-8.0:
  Raise E_WARNING on PHP related errors

Raise E_WARNING on PHP related errors

If Zip operations fails due to PHP error conditions before libzip even
has been called, there is no meaningful indication what failed; the
functions just return false, and the Zip status indicated that no error
did occur.  Therefore we raise `E_WARNING` in these cases.

Closes GH-6356.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Move stack overflow checks out of the loops

Move stack overflow checks out of the loops

Merge branch 'PHP-8.0'

* PHP-8.0:
  report about ZSTD compression availability
  only display libzip both headers/library versions if they differ

report about ZSTD compression availability

only display libzip both headers/library versions if they differ

Merge branch 'PHP-8.0'

* PHP-8.0:
  Backport preloading trait fixup fixes

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Backport preloading trait fixup fixes

Backport preloading trait fixup fixes

This cherry-picks 33969c2252b2c33a72c9039072af8862fd347a5f and
2effbfd8713936742ef46e6e85ce193b320ac005 from PHP-8.0.

The issues these commits fix could also manifest in PHP 7.4, and
a commenter on bug #80307 reports this this might indeed be
happening.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed incorrect invariant guard motion

Fixed incorrect invariant guard motion

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixup trait methods even if no traits are used

Fixup trait methods even if no traits are used

Trait methods might be non-trivially inherited, in which case we
may have to perform fixup in classes that do not directly use any
traits.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix multiple trait fixup

Fix multiple trait fixup

If a trait method is inherited, preloading trait fixup might be
performed on it multiple times. Usually this is fine, because
the opcodes pointer will have already been updated, and will thus
not be found in the xlat table.

However, it can happen that the new opcodes pointer is the same
as one of the old opcodes pointers, if the pointer has been reused
by the allocator. In this case we will look up the wrong op array
and overwrite the trait method with an unrelated trait method.

We fix this by indexing the xlat table not by the opcodes pointer,
but by the refcount pointer. The refcount pointer is not changed
during optimization, and accurately represents which op arrays
should use the same opcodes.

Fixes bug #80307. The test case does not reproduce the bug, because
this depends on a lot of "luck" with the allocator. The test case
merely illustrates a case where orig_op_array would have been NULL
in the original code.

Merge branch 'PHP-8.0'

* PHP-8.0:
  End output handlers in preload shutdown

End output handlers in preload shutdown

Same as in php_request_shutdown(), we need to end any active
output handlers, as these may no longer be safe to execute
lateron.

Merge branch 'PHP-8.0'

* PHP-8.0:
  getlastmod() can return false

getlastmod() can return false

At least this can happen during preloading.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix static variable in methods inheritance during preloading

Fix static variable in methods inheritance during preloading

This is now "bug compatible" with the normal behavior, and more
imporantly, does not crash :)

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #80266: parse_url silently drops port number 0

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #80266: parse_url silently drops port number 0

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #80266: parse_url silently drops port number 0

Fix #80266: parse_url silently drops port number 0

As of commit 81b2f3e[1], `parse_url()` accepts URLs with a zero port,
but does not report that port, what is wrong in hindsight.

Since the port number is stored as `unsigned short` there is no way to
distinguish between port zero and no port.  For BC reasons, we thus
introduce `parse_url_ex2()` which accepts an output parameter that
allows that distinction, and use the new function to fix the behavior.

The introduction of `parse_url_ex2()` has been suggested by Nikita.

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

Closes GH-6399.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Assert that references are not persisted

Assert that references are not persisted

There should not be any need to persist references, and it's unlikely
that persisting a reference will behave correctly at runtime, because
we don't have a concept of an immutable reference.

Fix parallel testing for IMAP

The script for the clean section was missing the $ Meta-character to only delete the default
mailbox when it's an exact match.

Reinstore parallel testing for more tests.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Don't xfail method_static_var.phpt

Don't xfail method_static_var.phpt

Let's test the current behavior here. It might not be right, but
it's long-standing behavior.

Nearly missed an assertion failure here because the test was
XFAILed...

Merge branch 'PHP-8.0'

* PHP-8.0:
  Don't disable opcache for generic tests

Don't disable opcache for generic tests

Even if the original issue only reproduces without opcache, we
should still allow running them with and without opcache.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix dynamic function definition in preload script

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix dynamic function definition in preload script

Fix dynamic function definition in preload script

We should use normal function renaming if the function is declared
during preloading itself, rather than afterwards.

This fixes a regression introduced by
68f80be9d1380de731930187250a7ed6b55ae196.

Make all IMAP fetch*() tests conflict with the default mailbox

hash: murmur: Initialize final hash explicitly

Signed-off-by: Anatol Belski <ab@php.net>

Merge branch 'PHP-8.0'

* PHP-8.0:
  Don't disable early binding during preloading script

Don't disable early binding during preloading script

We should only disable early binding during the opcache_compile_file()
calls, not inside the preloading script or anything it includes.
The right condition to check for is whether we compile the file
without execution, as declaring classes is "execution".

Merge branch 'PHP-8.0'

* PHP-8.0:
  Report parse errors during preloading

Report parse errors during preloading

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix use of type copy ctor when importing trait properties

Fix use of type copy ctor when importing trait properties

We shouldn't call the copy constructor inside the original type,
duh.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Allow unlinked classes when performing in_compilation variance check

Allow unlinked classes when performing in_compilation variance check

As preloading runs in in_compilation mode, we also need to allow
use of unlinked classes in lookup_class().

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix variance checks on resolved union types

Fix variance checks on resolved union types

This is a bit annoying: When preloading is used, types might be
resolved during inheritance checks, so we need to deal with CE
types rather than just NAME types everywhere.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Don't ignore internal classes during preloading

Don't ignore internal classes during preloading

When preloading, it's fine to make use of internal class information,
as we do not support Windows. It is also necessary to allow proper
variance checks against internal classes.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix persisting property info table with internal parent

Fix persisting property info table with internal parent

If the property info comes from an internal parent, we won't have
an xlat entry for it. Leave it alone in that case.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Preserve trait method alias name during preloading

Preserve trait method alias name during preloading

Modernize IMAP tests

Use constants when appropriate, drop some ouputs which depends on env vars

Enable parallel testing.

Closes GH-6380

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix another implicit function declaration in configure

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix another implicit function declaration in configure

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix another implicit function declaration in configure

Fix another implicit function declaration in configure

As mentioned on bug #80171. This one is in libtool.m4, might get
lost on libtool updates.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix typo in UPGRADING
  Improved error message for typed class properties with null as default value

Fix typo in UPGRADING

Closes GH-6395.

Improved error message for typed class properties with null as default value

Closes GH-6396.

Add test suite for ARMSCII-8 encoding

Fix mbstring support for ARMSCII-8

- Identify filter was completely wrong.
- Respect `mb_substitute_character` rather than converting invalid bytes to
  Unicode 0xFFFD (generic replacement character).
- Don't convert Unicode 0xFFFD to a valid ARMSCII-8 character.
- When converting ARMSCII-8 to ARMSCII-8, don't pass invalid bytes through
  silently.

Optimize (AND FIX) mb_check_encoding (cut execution time by 50%+)

Previously, `mb_check_encoding` did an awful lot of unneeded work. In order to
determine whether a string was valid or not, it would convert the whole string
into wchar (code points), which required dynamically allocating a (potentially
large) buffer. Then it would turn right around and convert that big 'ol buffer
of code points back to the original encoding again. Finally, it would check
whether any invalid bytes were detected during that long and onerous process.

The thing is, mbstring _already_ has machinery for detecting whether a string
is valid in a certain encoding or not, and it doesn't require copying any data
around or allocating buffers. Better yet, it can fail fast when an invalid byte
is found. Why not use it? It's sure a lot faster!

Further, the legacy code was also badly broken. Why? Because aside from
checking whether illegal characters were detected, it would also check whether
the conversion to and from wchars was lossless. But, some encodings have
more than one valid encoding for the same character. In such cases, it is
not possible to make the conversion to and from wchars lossless for every
valid character. So `mb_check_encoding` would actually reject good strings
in a lot of encodings!

Add test suite for KOI8-U encoding

Remove dead code from mbfilter_koi8u.c (and do general code cleanup)

All bytes are valid in KOI8-U encoding

Add test suite for KOI8-R encoding

Remove dead code from mbfilter_iso8859_{2,4,5,9,10,13,14,15,16}.c

...Plus some dead code related to ISO-8859-1.

Remove dead code from mbfilter_koi8r.c

All bytes are valid in KOI8-R encoding

Add test suite for CP850 encoding

Remove dead code from mbfilter_cp850.c (and do general code cleanup)

Since there are no invalid bytes in CP850, these `if` conditions will never
be true.

All bytes are valid in CP850 encoding

Add test suite for CP866 encoding

Remove dead code from mbfilter_cp866.c (and do general code cleanup)

Since there are no invalid bytes in CP866, these `if` conditions will never
be true.

All bytes are valid in CP866 encoding

Add test suite for CP1254 encoding

Fix mbstring support for CP1254 encoding

One funny thing: while the original author used Unicode 0xFFFD (generic
replacement character) for invalid bytes in CP1251 and CP1252, for CP1254
they used 0xFFFE, which is not a valid Unicode codepoint at all, but is a
reversed byte-order mark. Probably this was by mistake.

Anyways,

- Fixed identify filter, which was completely wrong.
- Don't convert Unicode 0xFFFE to a random (but valid) CP1254 byte.
- When converting CP1254 to CP1254, don't pass invalid bytes through silently.

Add test suite for CP1251 encoding

Fix mbstring support for CP1251 encoding

- Identify filter was as wrong as wrong can be.
- Invalid CP1251 byte 0x98 was converted to Unicode 0xFFFD (generic
  replacement character), rather than respecting `mb_substitute_character`.
- Unicode 0xFFFD was converted to some random CP1251 byte.
- When converting CP1251 to CP1251, don't pass invalid bytes through silently.

Test cases for mbstring encodings are less repetitive