Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:118444
To: internals@lists.php.net
References: <CAEKnhAEJ+bOnvES_5vmrOgP-PdXuw=M6AezY0Yh2Xi-nkx5RAA@mail.gmail.com>
Message-ID: <62FE6487.3000307@adviesenzo.nl>
Date: Thu, 18 Aug 2022 18:10:47 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:38.0) Gecko/20100101
 Thunderbird/38.7.0
MIME-Version: 1.0
In-Reply-To: <CAEKnhAEJ+bOnvES_5vmrOgP-PdXuw=M6AezY0Yh2Xi-nkx5RAA@mail.gmail.com>
Content-Type: multipart/alternative;
 boundary="------------000001080801050907030300"
Subject: Re: ***SPAM*** [PHP-DEV] Additional small features for 8.2
From: php-internals_nospam@adviesenzo.nl (Juliette Reinders Folmer)

--------------000001080801050907030300
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit

On 18-8-2022 15:46, Jakub Zelenka wrote:
> Make libxml_set_external_entity_loader() return the previous loader
> https://github.com/php/php-src/pull/7977

Based on the PR, this introduces a new function to PHP/the `libxml` 
extension. Doesn't that need an RFC ?

Note: I'm not opposed to the change as per the current implementation, 
just wondering if allowing this to go into PHP 8.2 without RFC is 
correct process-wise.

> Add openssl_cipher_key_length function
> https://github.com/php/php-src/pull/9368
Similar as the above, new function for the `openssl` extension.

I do think the two cases are different though as openssl is an optional 
extension, while libxml is enabled by default.

Smile,
Juliette

--------------000001080801050907030300--