Add get_resource_id() function

Behavior is same as for (int) $resource, just under a clearer
name. Also type-safe, in that the parameter actually needs to
be a resource.

Closes GH-5427.

Merge branch 'sse2_urlencode' of https://github.com/laruence/php-src

* 'sse2_urlencode' of https://github.com/laruence/php-src:
  drop use of extract_epi16
  Use SSE2 instructions do url_encode

Use supported printf format on Win64

Commit aaa1f90[1] dropped support for the custom `I` specifier, so we
must no longer use it for `ZEND_ADDR_FMT`; otherwise there are
segfaults when printing memory leaks, and maybe elsewhere.

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

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix default sendmail path when not found during build

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix default sendmail path when not found during build

Fix default sendmail path when not found during build

Closes GH-5548.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Don't check directory atime in lstat_stat_variation10.phpt test

Don't check directory atime in lstat_stat_variation10.phpt test

This is a funny one. I discovered that lstat_stat_variation10.phpt was failing every
now and again when the PHP test suite was run on my dev PC. The output from the failing
test showed that the atime (access time) of the directory created in the test was changing
between these lines:

    $old_stat = stat($dirname);
    clearstatcache();
    sleep(1);
    var_dump( is_dir($dirname) );
    $new_stat = stat($dirname);

Could is_dir() be accessing the directory and changing the atime? strace showed that is_dir
was only issuing a single stat() syscall. Could stat() change the atime? No, no, that would
just be perverse. Nobody would be stupid enough to implement the kernel in that way.

Checked the kernel source, found that the function called when atime needs to be updated
appears to be touch_atime(). Broke out the BCC kernel tracing tools and ran this one
while running the flaky test case in a loop:

    sudo trace -I<kernel src dir>/include/linux/path.h -I<same>/include/linux/dcache.h 'touch_atime(struct path *path) "%s", path->dentry->d_name.name'

Inspecting the results showed that something called "git_thread" was occcasionally updating
the atime on the directory in question!! What on earth...???

The PID shown by trace revealed that this was a background thread for Sublime Text 3.
Sublime now has git integration and shows when there are untracked or modified files. It
seems that it uses a background thread to regularly scan the project directory and look
for new and modified files. This was causing the atime to change.

Even though other developers may not be running ST3, there are any number of reasons why
a background process might recurse through various directories and could cause the atime
to change unexpectedly. Therefore, update the test case so it doesn't fail in such cases.

Closes GH-5553.

Fix use-after-free in sysvsem

Add `pkg-config` to the build list

Closes GH-5550.

[ci skip]

Fix bug68996.phpt test

The function name gets printed here now, with illegal chars
substituted in the arg. This slipped through previously, because
the test only runs under USE_ZEND_ALLOC=0. We remove that check
as well, because the test behaves the same regardless of which
allocator is used, even if the original bug only reproduces under
USE_ZEND_ALLOC=0.

drop use of extract_epi16

Skip test if bigint type is unsupported by active TDS version

Set non-default error mode to maintain expected behavior

Update FreeTDS version detection related to bug #72969

Make float to string casts locale-independent

From now on, float to string casting will always behave locale-independently.
RFC: https://wiki.php.net/rfc/locale_independent_float_to_string
Closes GH-5224

Co-authored-by: George Peter Banyard <girgias@php.net>

Use SSE2 instructions do url_encode

cleanup

Change parameter meaning.

Use proper "cost"

Avoid loop throgh side_exit from the first trace instruction.

Fixed tracing JIT for CALL VM without global register variables

Ensure ctype_string is NULL for C locale

We already document that this is the case, but currently it's only
true if setlocale() has not been called. Make sure ctype_string is
always NULL, even with an explicit "C" locale call, so we can
more efficiently check whether we are in the "C" locale.

Closes GH-5542.

Fix test file encoding

The mb_http_input_pass.phpt was intended to use the same encoding
as mb_http_input.phpt, not UTF-8.

More accurate tracing JIT for RETURN with unknown return address

Rename locale_string to ctype_string

To make it more obvious that this only refers to the LC_CTYPE
locale.

Remove redundant htmlentities() tests

Test 04 and 15 are the same as 02 and 03, just for different
encodings. They don't add value, but their execution depends
on available locales, so they're easy to miss...

Folder marks

Really fix test case

Fix test case

Evaluate str_starts_with and str_ends_with on constants

When both arguments are strings,
this is guaranteed not to emit notices.

Closes GH-5539

Don't respect mbstring.internal_encoding in htmlentities()

htmlentities() has nothing to do with mbstring and should not
depend on its ini settings. It should only respect the global
default_charset and internal_encoding settings. This is exactly
why they were introduced...

Add quiet parameter to internal HTML entities API

In some places, we need to make sure that no warnings are thrown
due to unknown encoding. The error reporting code tried to avoid
this by determining a "safe charset", but this introduces subtle
discrepancies in which charset is picked (normally
internal_encoding takes precedence). Avoid this by suppressing
the warning in the first place.

While here, use the fallback logic to print error messages with
substitution characters more consistently, to avoid skipping
parts of the error message entirely.

Clean up determine_charset() implementation

And drop code related to locale-based charset guessing,
which is no longer in use.

Throw warning if invalid internal_encoding ini is specified

Prevent taking side traces for exceptional cases. Always exit to VM interpreter.

Remove no_language from mbfl_string

This is not actually used for anything and just causes confusion.

Only allow "pass" as input/output encoding

"pass" is not a real encoding, it just means "don't perform any
conversion". Using it as an internal encoding or passing it to
any of the mbstring() function will not work (and on master commonly
assert).

Rename the recently introduced Sysvsem class to SysvSemaphore

Return false from failed mb_convert_variables()

If we fail to detect the encoding return false, just like
mb_convert_encoding() does, and the implementation here clearly
intended. Previously the "pass" pseudo-incoding was returned.

Enable test on Windows

Fixed false alarm about tmp_name maybe unitialized

Fixed comment

Fix assertion failure when failing to detect encoding

Looks like prior to 7.3 this just passed the original string
through. Since 7.3 it returns false. Let's stick with that
behavior.

Avoid Program Counter guard in side trace started from Fake Init Fcall guard.

Use int|string Fast ZPP macro in Reflection

Moreover, throw a more appropriate ValueError in case the integer
position provided is less than 0.

Closes GH-5513

Initial support for IS_INDIRECT. Avoid type guards for IS_INDIRECT.

Promote warning to exception in ZipArchive::extractTo()

Closes GH-5516

Fix UNKNOWN default values in ext/spl

Fix UNKNOWN default values in ext/sqlite3

Fix UNKNOWN default values in ext/xml

Fix UNKNOWN default values in ext/xsl

Fix UNKNOWN default values in ext/zip

Fix ZPP order in ext/zip

Drop BF_ASM constant as it's always 0

Closes GH-5531

Remove out-dated comments

Convert resource to object in ext/sysvsem

Closes GH-5508

Careful cleanup

add myself as enchant maintainer

Fix enchant stub

enchant: bump license version

Add zend_array_release() function

To complement zend_string_release() and zend_object_release().

Revert "Move declaration at top of the block"

This reverts commit b56f20385089703de7a3c9d89a9de4f9d14f56a3.

We use C99 now, this is not needed anymore.

Move declaration at top of the block

Use correct variable

This seems to stem from a copy paste, however there is no object variable in this function

Fix UNKNOWN default values in various extensions

Closes GH-5514

Make numeric operations on resources, arrays and objects type errors

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

Closes GH-5331.

Add str_starts_with() and str_ends_with()

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

Closes GH-5300.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79566: Private SHM is not private on Windows

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79566: Private SHM is not private on Windows

Fix #79566: Private SHM is not private on Windows

We map the POSIX semantics of `IPC_PRIVATE` by creating unnamed file
mapping objects on Windows.  While that is not particularly useful for
ext/shmop, which is the only bundled extension which uses `shmget()`,
it may be useful for external extensions.

Fix mbfl default allocators

Forgot to remove the persistent allocators from here.

Spec mbfl allocators as infallible

And remove all NULL checks.

Remove persistent allocators from libmbfl

These functions are not used, and I don't think we have any plans
to ever use them.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79561: dns_get_record() fails with DNS_ALL

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79561: dns_get_record() fails with DNS_ALL

Fix #79561: dns_get_record() fails with DNS_ALL

Since Windows has its own definitions of the `PHP_DNS_*` macros, we
have to use these when registering the PHP constants.

Change the default PDO error mode to exceptions

According to <https://www.php.net/manual/en/pdo.error-handling.php>.

Perform isupper check using sse2 as well

Rather than just vectorizing the lowering, also vectorize the
check for uppercase characters, using the same method.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79548

Fixed bug #79548

When duplicating user functions with static variables, make sure
that we init a new map ptr slot for the static variables.

Extract one more function from proc_open() implementation

Clean up proc_open() implementation

Closes GH-5507.

Remove unused is_persistent flag

I dropped the assignment to it before, but did not drop the
struct member.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79535

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79535

Fixed bug #79535

We did not allocate a cache slot for FETCH_CLASS. This is already
fixed on newer PHP versions.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix i386 build on Azure

Fix i386 build on Azure

Disabling postgresql entirely for now, because I can't figure out
how to fix it. Something broke big time with i386 packages on
Azure pipelines.

Use ZPP int|string and add ValueError for Windows codepages

Closes GH-5517

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add basic sapi_windows_cp_conv() test

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Add basic sapi_windows_cp_conv() test

Add basic sapi_windows_cp_conv() test

This function is lacking any tests so far.

Use libenchant2 if available on Windows

This is basically the same as commit 3427545[1], but for Windows.

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

Use libenchant-2 when available

Fix #79467: data:// wrappers are writable

Despite the docs claiming that data: wrappers would not be writable[1],
they are implemented as writing to a memory stream.  That does not seem
to be particularly sensible, so we disallow writing altogether.

[1] <https://www.php.net/manual/en/wrappers.data.php#refsect1-wrappers.data-options>

Use ZPP check for string|int|null arguments in array_column()

Add Fast ZPP string|int type check

Don't raise bogus warning if writing completely failed

Remove is_persistent flag from proc_open implementation

We don't support persistent proc_open handles and have no plan
to suppor them. The mixture of persistent and non-persistent
allocations functions in this code is somewhat confusing to read.