Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:103932
MIME-Version: 1.0
References: <CAOwTYAsePeRLZZVK=QKftoOtug2_F=5bsti56z2mJfBXP0u4rw@mail.gmail.com>
 <DM6PR02MB5594D2DE0A166B13024E44B9AC910@DM6PR02MB5594.namprd02.prod.outlook.com>
 <CAOwTYAtr=eoRxAR4yOT_a-zuw_m9_wvNU-01eX3AyyQd5Rgw2w@mail.gmail.com>
 <DM6PR02MB5594464D33F2A0A02D747D6BAC910@DM6PR02MB5594.namprd02.prod.outlook.com>
 <CAOwTYAvnRaszSVi9Ua0AGdnMP5YCRjV7UGAnyk9DJ+qfXLG26Q@mail.gmail.com>
 <CAJHtznySqZY2WrU32faAcOE+tuo8v716BHmZtynCqRseWn1-cQ@mail.gmail.com>
 <CAOwTYAuTaLha3CAFPyj6xsXgKfhROgbqLMyhb-6R+4j5wy_PYw@mail.gmail.com>
 <13d4f2b0-6d4f-d6de-e61f-4fb782da82bf@zend.com> <CAJHtznxyGKaBWMr6j5gqsVHRrHCNODA0XvPBURuGoGMQUwtfAQ@mail.gmail.com>
In-Reply-To: <CAJHtznxyGKaBWMr6j5gqsVHRrHCNODA0XvPBURuGoGMQUwtfAQ@mail.gmail.com>
Date: Thu, 31 Jan 2019 17:23:40 +0100
Message-ID: <CAOwTYAsM-J_OCUxeEkgB5gS53ya0QmA6jV8PDjpBeL7-OeyeRQ@mail.gmail.com>
To: Zeev Suraski <zeev@php.net>
Cc: Dmitry Stogov <dmitry@zend.com>, Zeev Suraski <zeev@zend.com>, 
	"internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000b7f90b0580c373da"
Subject: Re: [PHP-DEV] RFC Abolish Narrow Margins
From: krakjoe@gmail.com (Joe Watkins)

--000000000000b7f90b0580c373da
Content-Type: text/plain; charset="UTF-8"

Hi Zeev, Dmitry,

It is not my only concern, I'm grateful for the clarification whatever.

These are your actual words from the RFC:

> This works, but this functionality is not supported on all libffi
platforms, it is not efficient and leaks resources by the end of request.
It's recommended to minimize the usage of PHP callbacks.

To me, a foreign function interface where one side of the interface sounds
broken and inefficient, according to the author, is scary. I've never
worked with libffi, so don't know if that's normal ... but I think you can
understand my thoughts here.

Aside from whatever technical issues I may have misinterpreted or
misidentified, these are not my main objections to it being merged.

Cheers
Joe

On Thu, 31 Jan 2019 at 17:20, Zeev Suraski <zeev@php.net> wrote:

>
>
> On Thu, Jan 31, 2019 at 6:09 PM Dmitry Stogov <dmitry@zend.com> wrote:
>
>> The fact that FFI may leak, is because it uses "unmanaged memory" (like
>> in real C). PHP reference counting and GC just can't handle all the
>> cases. It's impossible to automatically know what to do with C pointer,
>> when we get it from function or another structure. In this case
>> programmer have to care about freeing the pointer themselves (like in C).
>>
>> This obviously doesn't count as 'leaks'.  If it did, we could say that
> the Zend Extension API leaks, as it's completely up to the developer to
> handle memory management.
>
> Joe - if that's your concern, I think it's fair to say it's invalid and
> you have nothing to worry about.
>
> Zeev
>

--000000000000b7f90b0580c373da--