Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:96004 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 66676 invoked from network); 15 Sep 2016 00:10:50 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 15 Sep 2016 00:10:50 -0000 X-Host-Fingerprint: 2.221.201.203 unknown Received: from [2.221.201.203] ([2.221.201.203:12853] helo=localhost.localdomain) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 7E/BB-21040-807E9D75 for ; Wed, 14 Sep 2016 20:10:49 -0400 Message-ID: <7E.BB.21040.807E9D75@pb1.pair.com> To: internals@lists.php.net References: Date: Thu, 15 Sep 2016 01:10:43 +0100 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:43.0) Gecko/20100101 Firefox/43.0 SeaMonkey/2.40 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Posted-By: 2.221.201.203 Subject: Re: [PHP-DEV] Follow up to "The death of `#ifndef FAST_ZPP`?" From: ajf@ajf.me (Andrea Faulds) Hi Pierre, Pierre Joye wrote: > > Thanks you :) > > Please add this note to UPGRADING.INTERNALS as well : Well, I was wondering about that, but I don't really know where it should be noted. It's not a user-facing change, in fact nobody was disabling FAST_ZPP anyway, so it couldn't be one. Therefore, it shouldn't be noted in NEWS. And UPGRADING.INTERNALS is an upgrading guide used between major or minor releases. This is a patch change that affects not only master, but 7.1 and 7.0. So it isn't suitable for that, either. Plus, UPGRADING.INTERNALS has no existing mention of Fast ZPP. So, it doesn't seem to go anywhere, unfortunately. :/ Thanks. -- Andrea Faulds https://ajf.me/