Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:105553
MIME-Version: 1.0
References: <b6f883a4-eb6b-f151-8867-e41ce00b7ead@gmx.de>
In-Reply-To: <b6f883a4-eb6b-f151-8867-e41ce00b7ead@gmx.de>
Date: Wed, 1 May 2019 23:13:51 -0500
Message-ID: <CAESVnVp8955r4ekyx4XBiKD4FzQ5ROm=27W600w-c7xwJJSngA@mail.gmail.com>
To: "Christoph M. Becker" <cmbecker69@gmx.de>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="0000000000003a5a490587dfdd98"
Subject: Re: [PHP-DEV] Remove $age parameter of curl_version()
From: pollita@php.net (Sara Golemon)

--0000000000003a5a490587dfdd98
Content-Type: text/plain; charset="UTF-8"

On Wed, May 1, 2019 at 12:18 PM Christoph M. Becker <cmbecker69@gmx.de>
wrote:

>
> curl_version()[1] (of ext/curl) makes curl_version_info()[2] (of
> libcurl) available to PHP userland.  The latter requires to pass an age
> argument which usually is CURLVERSION_NOW, so that the information
> returned by the runtime matches the declarations used during compile
> time.  For C programs it is simply necessary to pass this information,
> and it might make sense to pass something else than CURLVERSION_NOW, but
> the PHP wrapper assumes that the return value of curl_version_info()
> matches the compile time declarations anyway, so passing anything else
> might give bad results.
>
> Wow... yeah. That's an example of being far too idiomatic with the
bindings. I didn't even know we accepted that arg.  Kill it with fire.

-Sara

--0000000000003a5a490587dfdd98--