Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:118022
Message-ID: <6d24947d-e004-1354-a608-b344b4f151a5@gmail.com>
Date: Mon, 20 Jun 2022 17:17:35 +0100
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.10.0
Content-Language: en-GB
To: internals@lists.php.net
References: <CAOfKkdRzoo7JuD07F8nkZBYvS5eDMiV2r6cNV-FgeLxBrdCsiQ@mail.gmail.com>
 <CAESVnVpzqT0xtEShqQp+FPXt2DQjeAXpkSVZTACAC0CW=8FBBg@mail.gmail.com>
 <CAOfKkdTu1f6jJ2HwGjOnBR2sO6P6qdOrjH1ZzumwLzf1syXa6Q@mail.gmail.com>
 <76FA5161-16C3-4570-9EF8-B75DF2E6530A@gmail.com>
 <CAOfKkdQGX5Lf5-jKyKnh5im9YYd-pKPx=AxnLgk9=tBoJ37E9Q@mail.gmail.com>
 <5758628b-197f-000c-bfbf-8b6884b11223@gmx.de>
In-Reply-To: <5758628b-197f-000c-bfbf-8b6884b11223@gmx.de>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] Discussion about new Curl URL API and ext/curl
 improvements
From: rowan.collins@gmail.com (Rowan Tommins)

On 20/06/2022 14:44, Christoph M. Becker wrote:
> Given that designing a new OO API obviously isn't uncontroversial, and
> that there are only four weeks left till feature freeze, it might be
> best to go with a simple procedural API for PHP 8.2 only.


I haven't read back through the thread, but my impression was that *for 
the curl URL facility specifically* the opposite was the case: a simple 
object with no procedural equivalent would be everyone's preference. 
CurlFile provides enough of a precedent for adding that IMO.

Either way, we have to agree on some naming details, so I've left some 
comments on the PR.

(By the way, the PR you linked is to merge into your own fork's master, 
not the actual central php-src repo. Not sure if that was deliberate.)

Regards,

-- 
Rowan Tommins
[IMSoP]