Use Z_PARAM_OBJ macros when zval isn't needed

In some cases, like spl_object_id, the code is simpler but equally efficient
after optimizations.

In other cases, like get_mangled_object_vars(), the compiler can't infer that
the object in the zval won't change.

Closes GH-6567

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix parameter name

Fix parameter name

Levenshtein is about insertion, replacement and deletion.

Closes GH-6560.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix build

Fix build

We cannot `RETURN_THROWS()` here, since `return_value` is not defined.

Merge branch 'PHP-8.0'

Use RETURN_THROWS() in a few places

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix ReflectionClass::getConstants() stub

Fix ReflectionClass::getConstants() stub

If `zval_update_constant_ex()` fails, an exception has already been
thrown, so we clarify that in the implementation as well.

Closes GH-6557.

Merge branch 'PHP-8.0'

* PHP-8.0:
  [ci skip] Fix pgsql constant names in UPGRADING

[ci skip] Fix pgsql constant names in UPGRADING

Merge branch 'PHP-8.0'

Fix refentry attribute replacing

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #80560: Strings containing only a base prefix return 0 object

Fix #80560: Strings containing only a base prefix return 0 object

Closes GH-6549.

Remove superfluous return statement.

Remove redundant check in SplObjectStorage->count

Merge branch 'PHP-8.0'

* PHP-8.0:
  Initialize EX(call)->func by single instruction
  Reuse value stored in %r0 instead of immediate operand

Initialize EX(call)->func by single instruction

Reuse value stored in %r0 instead of immediate operand

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix ffi stubs

Fix ffi stubs

Closes GH-6543.

Merge branch 'PHP-8.0'

Fix return type of DateTimeImmutable::__set_state()

Add support for generating methodsynopses from stubs

Closes GH-6367

Merge branch 'PHP-8.0'

* PHP-8.0:
  Bump minimum re2c version to 0.13.7

Bump minimum re2c version to 0.13.7

Closes GH-6516.

Merge branch 'PHP-8.0'

* PHP-8.0:
  php_formatted_print() throws on failure

php_formatted_print() throws on failure

Closes GH-6542.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Replace sort() function calls with ksort() in basic ksort test

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Replace sort() function calls with ksort() in basic ksort test

Replace sort() function calls with ksort() in basic ksort test

Closes GH-6541.

Merge branch 'PHP-8.0'

* PHP-8.0:
  DateTime*::getOffset() no longer returns false as of PHP 8.0.0

DateTime*::getOffset() no longer returns false as of PHP 8.0.0

Cf. <https://github.com/php/doc-en/pull/282>.

Closes GH-6539.

[skip ci] fix typo in NEWS

Merge branch 'PHP-8.0'

* PHP-8.0:
  Eliminate redundand comparison insructions

Eliminate redundand comparison insructions

Merge branch 'PHP-8.0'

Fix the signature of FFI::typeof() and FFI::memcmp()

Add a few other RETURN_THROWS()

Fix the signature of FFI::sizeof() and FFI::alignof()

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix PDO_OCI test

Fix PDO_OCI test

Fix PDO_OCI build

Update hash for IMAP stubs after adding final qualifier

IMAP: Declare `IMAPConnection` class as `final` (stub+test)

Updates the `IMAPConnection` class stub to make sure it has the `final` flag, and adds a test to verify it.

IMAP: Disallow direct `new IMAPConnection()` construct

Disallows constructing an `IMAPConnection` class directly with `new IMAPConnection` construct, by throwing an `Error` exception if attempted.
`imap_open` is still the only way to create `IMAPConnection` objects.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Add regression test for bug #76770

Add regression test for bug #76770

Closes GH-3514.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #80384: limit read buffer size

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #80384: limit read buffer size

Fix #80384: limit read buffer size

In the case of a stream with no filters, php_stream_fill_read_buffer
only reads stream->chunk_size into the read buffer. If the stream has
filters attached, it could unnecessarily buffer a large amount of data.

With this change, php_stream_fill_read_buffer only proceeds until either
the requested size or stream->chunk_size is available in the read buffer.

Co-authored-by: Christoph M. Becker <cmbecker69@gmx.de>
Closes GH-6444.

Support native types in PDO SQLite

Return integers and floats as native types if possible. As usual,
the old behavior can be restored by enabling ATTR_STRINGIFY_FETCHES.

Fixes bug #38334.

FTP: Disallow direct `FTPConnection` construction

Similar to other resource to object migrations, `FTPConnection` class is not allowed to be constructed with `new FTPConnection`.
Related to b4503fbf882e490f16d85915e83173bd1e414e84.

Closes GH-6533.

EXPECTF imap_errors() test's authentication error

Make imap_errors() test no hang

Instead of attempting to connect with an invalid password, connect as an anonymous user.
As this is disabled in our Dovecot configuration, this generates an error, and fails fast.

Optimize Dovecot configuration for IMAP tests

Closes GH-6531

Rewrite PDO result binding

Instead of requiring the type to be determined in advance by the
describer function and then requiring get_col to return a buffer
of appropriate type, allow get_col to return an arbitrary zval.
See UPGRADING.INTERNALS for a more detailed description of the
change.

This makes the result fetching simpler, more efficient and more
flexible. The general possibility already existed via the special
PDO_PARAM_ZVAL type, but the usage was very inconvenient and/or
inefficient. Now it's possible to easily implement behavior like
"return int if it fits, otherwise string" and to avoid any kind
of complex management of temporary buffers.

This also fixes bug #40913 (our second highest voted bug of all
time, for some reason). PARAM_LOB result bindings will now
consistently return a stream resource, independently of the used
database driver.

I've tried my best to update all PDO drivers for this change, but
some of the changes may be broken, as I cannot test or even build
some of these drivers (in particular PDO dblib and PDO oci).
Fixes are appreciated -- a working CI setup would be even more
appreciated ;)

Don't use explicitly nullable column in bug_73234.phpt

Firebird doesn't support this. It is nullable by default.

The test still fails, but now for a legitimate reason...

Add CONFLICTS file for PDO firebird tests

Fix bug_69356.phpt for firebird

Firebird does not support selecting without a table. Use the same
code we use elsewhere, which adds "FROM DUAL" or "FROM RDS$DATABASE"
as necessary.

PDO Firebird: Use recreate table

Firebird does not have "if exists", and the switch to exception
error mode broke the suppressed exec calls.

Firebird does have a "recreate table" command that effective
perform a "drop table if exists" beforehand.

Suppress zend_signals warnings if pdo_firebird loaded

Add IMAP resource to object conversion to UPGRADING

Convert IMAP resource to object

Closes GH-6418

Build PDO Firebird on Azure

Once again, this only builds the driver, all tests will be skipped.

Closes GH-6529.

Show slow SKIPIF sections as well

PDO Firebird: Fix uninitialized var warning

Build PDO odbc on azure

We already install the necessary libs for the odbc extension, so
we may as well build PDO ODBC.

As we don't setup an ODBC server, the tests will be skipped.

Close GH-6528.

PDO: Honor ATTR_STRINGIFY_FETCHES for booleans

Of the important PDO drivers, this affects only PDO PgSQL, as
both MySQL and SQLite do not return native boolean types.

Merge branch 'PHP-8.0' into master

Fixed bug #80376 (last day of the month causes runway cpu usage)

Fixed bug #80376 (last day of the month causes runway cpu usage)

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix bug #80537

Fix bug #80537

This is an unavoidable breaking change to both the type and
parameter name.

The assertion that was supposed to prevent this was overly lax
and accepted any object type for string parameters.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Description typo

Description typo

Merge branch 'PHP-8.0'

* PHP-8.0:
  Bundle PECL OCI8 3.0.1

Bundle PECL OCI8 3.0.1

Merge branch 'PHP-8.0'

* PHP-8.0:
  Retrofit NEWS

Retrofit NEWS

Merge branch 'PHP-8.0'

* PHP-8.0:
  Revert "DateTime:: and DateTimeImmutable::getTimestamp() may return false"

Revert "DateTime:: and DateTimeImmutable::getTimestamp() may return false"

This reverts commit b67c23218950eae0a153b093dcefbe14b3b40901.

Merge branch 'PHP-8.0'

* PHP-8.0:
  DateTime:: and DateTimeImmutable::getTimestamp() may return false

DateTime:: and DateTimeImmutable::getTimestamp() may return false

Convert file_info resources to objects

Besides our general desire to get rid of the legacy resource types,
this is particularly appealing for fileinfo, because there are already
respective objects.

Closes GH-5987.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix ext/date stubs

Fix ext/date stubs

Closes GH-6523.

Fix leak

Back memory stream by a zend_string

This allows reusing an existing zend_string inside a memory stream
without reallocating. For non-readonly streams, the string will
only get separated on write.

Remove unused smax member from memory stream

Remove unused pdo_pgsql_column member

Try to fix azure pipelines config

Is this space the problem?

Add basic libmysqlclient CI job

This tests that mysqli and pdo_mysql build against libmysqlclient,
and that tests pass for pdo_mysql. mysqli has too many test failures.

This is not an officially supported configuration.

Move fetch_all implementation out of mysqlnd

There doesn't seem to be any compelling reason to implement this
in mysqlnd rather than mysqli. It's just a loop over fetch_into.

This makes the function available under libmysqlclient as well,
and thus fixes bug #79372.

Make mysqli_stmt_next_result available under libmysql

The libmysqlclient version we require supports mysql_stmt_next_result.

PDO MySQL: Make test libmysql compatible

Merge branch 'PHP-8.0'

* PHP-8.0:
  PDO MySQL: Handle boolean parameter binding with libmysql

PDO MySQL: Handle boolean parameter binding with libmysql

Previously boolean parameters were simply silently ignored...

Merge branch 'PHP-8.0'

* PHP-8.0:
  MySQLnd: Support cursors in store/get result