Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:86776
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: unknown (pb1.pair.com: domain php.net does not designate 212.27.42.2 as permitted sender)
Reply-To: <francois@php.net>
To: <internals@lists.php.net>
Date: Mon, 22 Jun 2015 03:50:23 +0200
Message-ID: <001a01d0ac8d$cdeecd50$69cc67f0$@php.net>
MIME-Version: 1.0
Content-Type: text/plain;
	charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Thread-Index: AdCsfqoIxDyr4s0mQSiFLvswmheM2w==
Content-Language: fr
Subject: [PROPOSAL] zend_string API changes and additions
From: francois@php.net (=?iso-8859-1?Q?Fran=E7ois_Laupretre?=)

Hi,

I'd like your comments on a PR =
(https://github.com/php/php-src/pull/1353) I
just wrote to propose two changes to the zend_string API. Both are =
different
but, as they apply to the same file, it was easier to merge them the =
same PR
:

1. Suppress the 'persistent' arg from zend_string =
realloc/extend/truncate
functions

This argument is useless because :

    - perealloc() cannot re-allocate from non-persistent memory to
persistent or from persistent to non-persistent. So, the 'persistent'
argument value we give to the zend_string_realloc() function is totally
constrained : it *must* be the same as the value provided when the
zend_string was created.

    - This information is present in the zend_string structure as
(GC_FLAGS(s) & IS_STR_PERSISTENT).

So, the 'persistent' argument is useless and can only lead to failures.
That's why I propose we remove it from the API as soon as possible =
(7.0.0
alpha2 ?). It creates a small BC break for extension maintainers, but I
think it's still better than pulling this behind us during years. As an
indication of the BC break for extension maintainers, the whole core
distribution contains about 50 calls to one of these functions (PR =
includes
fixes).

2. Add a set of zend_string_append() functions

Added functions to append to a zend_string from :

    - another zend_string,
    - a pair (char *, len),
    - a literal string,
    - or a null-terminated string (using strlen()).

IMHO, these features are common enough to be part of the API.

Regards

Fran=E7ois