Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:79391
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.216.180 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <790a8d7299a7cda01ee76d8293b7d5af.squirrel@webmail.klapt.com>
References: <790a8d7299a7cda01ee76d8293b7d5af.squirrel@webmail.klapt.com>
Date: Wed, 3 Dec 2014 12:36:39 +1100
Message-ID: <CAEZPtU5TEo8w6gwHpEJSPLtirPD9eiMGdCNG=YosQLPvN894ow@mail.gmail.com>
To: Anatol Belski <anatol.php@belski.net>
Cc: PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [PHP-DEV] [RFC] Native TLS
From: pierre.php@gmail.com (Pierre Joye)

hi Anatol!

On Fri, Nov 28, 2014 at 2:54 AM, Anatol Belski <anatol.php@belski.net> wrote:
> Hi,
>
> this is a long spoken topic which is now embodied in
> https://wiki.php.net/rfc/native-tls . A preliminary implementation is
> there as well, thus we can discuss it.

Awesome work as usual!


As other mentioned, we should get rid of the TSRM_* macros for
declaratons/calls. Having them as placebo for the sake of having them
and reduce the amount of changes brought by this patch make little
sense to me. Master requires now a lot of changes anyway, we may as
well just kill that while being at it, easy move. Maybe add a vote
option to either keep the placebo (less code change) or remove it
(easier in the long term, no more TS build break, or fewer).

Cheers,
-- 
Pierre

@pierrejoye | http://www.libgd.org