Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:74563
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain zend.com from 209.85.220.172 cause and error)
MIME-Version: 1.0
In-Reply-To: <alpine.DEB.2.10.1405281247260.5108@whisky.home.derickrethans.nl>
References: <CA+9eiLuknJ7am5jU4WBN7fGXUcZtbb6axsji8ZnydCJ4P-O9hw@mail.gmail.com>
	<79B69A20-15AD-4C07-B026-61C2C9DEBBE5@ajf.me>
	<1400851698.2870.9315.camel@guybrush>
	<alpine.DEB.2.10.1405281247260.5108@whisky.home.derickrethans.nl>
Date: Wed, 28 May 2014 16:13:08 +0400
Message-ID: <CA+9eiLt_sgNVGRT-m+70SNjztk4fUWVS329hsiNzjPN9z+3Q7A@mail.gmail.com>
To: Derick Rethans <derick@php.net>
Cc: =?UTF-8?Q?Johannes_Schl=C3=BCter?= <johannes@schlueters.de>, 
	Andrea Faulds <ajf@ajf.me>, PHP Internals <internals@lists.php.net>, Bob Weinand <bobwei9@hotmail.com>, 
	Stas Malyshev <smalyshev@sugarcrm.com>, Zeev Suraski <zeev@zend.com>, Xinchen Hui <xinchen.h@zend.com>, 
	Andi Gutmans <andi@zend.com>, Nikita Popov <nikita.ppv@gmail.com>
Content-Type: multipart/alternative; boundary=089e0122f3324c18eb04fa74bcca
Subject: Re: [PHP-DEV] [RFC] Fast Parameter Parsing API
From: dmitry@zend.com (Dmitry Stogov)

--089e0122f3324c18eb04fa74bcca
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

This API was designed to solve the existing bottleneck (5% CPU time lose).
It's not proposed to be used in all functions, only in few really affected.

Anyone may have their own opinion about readability (someones like Perl,
others Ada).

I appeal to think not about subjective readability, but about the cost of
ownership for @internals and PHP users.

Thanks. Dmitry.


On Wed, May 28, 2014 at 3:49 PM, Derick Rethans <derick@php.net> wrote:

> On Fri, 23 May 2014, Johannes Schl=C3=BCter wrote:
>
> > On Fri, 2014-05-23 at 13:52 +0100, Andrea Faulds wrote:
> > > On 23 May 2014, at 12:36, Dmitry Stogov <dmitry@zend.com> wrote:
> > >
> > > > Please take a look at https://wiki.php.net/rfc/fast_zpp
> > > >
> > > > Thanks. Dmitry.
> > >
> > > This looks like a great idea. I like how it=E2=80=99s not only faster=
, but
> > > it seems to be more readable.
> >
> > More readable? - I feel exactly the opposite. Yes, we get rid of void
> > pointers but use a pattern easy to understand. This looks like macro
> > hell.
>
> I don't find it more readable either. From a single line, it goes to 7,
> and what if there are three different zpp calls (like in
> https://github.com/php/php-src/blob/master/ext/date/php_date.c#L4393) ?
>
> I know the old API is not going away, but I'm not just sure about this
> one...
>
>
> > If we break APIs anyways: Can't we go for C++ which allows providing
> > high level APIs with high performance ...
>
> no :-)
>
> cheers,
> Derick
>
> --
> http://derickrethans.nl | http://xdebug.org
> Like Xdebug? Consider a donation: http://xdebug.org/donate.php
> twitter: @derickr and @xdebug
> Posted with an email client that doesn't mangle email: alpine

--089e0122f3324c18eb04fa74bcca--