Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:70174
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.215.45 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <528734A3.5070001@php.net>
References: <5A.40.45833.7E3F3825@pb1.pair.com>
	<BC8FB99F-6EAD-43AB-AC25-56AFB30A2560@zend.com>
	<528734A3.5070001@php.net>
Date: Sat, 16 Nov 2013 16:33:10 +0100
Message-ID: <CAEZPtU61DxGbO5f_4D3-TXgzpi18KJsB-oUEJqwbqVGg-55f4Q@mail.gmail.com>
To: Joe Watkins <krakjoe@php.net>
Cc: Andi Gutmans <andi@zend.com>, PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a11c264a43d48b404eb4d081f
Subject: Re: [PHP-DEV] patch for a debugger in the making ...
From: pierre.php@gmail.com (Pierre Joye)

--001a11c264a43d48b404eb4d081f
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Hi Joe,

On Nov 16, 2013 10:02 AM, "Joe Watkins" <krakjoe@php.net> wrote:
>
> On 11/16/2013 03:01 AM, Andi Gutmans wrote:
>>
>> Hi Joe,
>>
>> Congrats on launching this debugger interface.
>> I=E2=80=99ve actually been thinking for some time that it=E2=80=99d be i=
ncredibly useful
to have something like this in the core distribution of PHP.
>>
>> Is this something that would be interesting for you to work towards?
What do others think?
>>
>> Andi
>>
>> P.S. - It wasn=E2=80=99t clear to me what the =E2=80=9Cquiet=E2=80=9D fl=
ag does but I would air
on the side of not showing opcodes by default but have an option where they
are shown, and in that case, potentially show even more detail [that can
easily be contributed.
>>
>>
>> On Nov 13, 2013, at 1:49 PM, Joe Watkins <krakjoe@php.net> wrote:
>>
>>> Evening All,
>>>
>>>         A couple of us are working on a debugging platform, in order to
deploy on windows we require a few more functions to be declared ZEND_API:
>>>
>>>         https://gist.github.com/krakjoe/7456833
>>>
>>>         If it's not going to cause any disruption, could we merge that
into master please ??
>>>
>>> Cheers
>>> Joe
>>>
>>> --
>>> PHP Internals - PHP Runtime Development Mailing List
>>> To unsubscribe, visit: http://www.php.net/unsub.php
>>>
>>
>
> Morning Andi,
>
>         Thanks :)
>
>         Pending the opinion of everyone else; but YES, definitely
interested in working toward that ...
>
>         We do indeed air on the side of caution with quietness enabled by
default. The only time quietness is ignored is while stepping, when it's
useful to show the oplog regardless. We do intend for the oplog to be a
decent representation of the opline, I've had some luck representing op
arrays in a human understandable way while working on explain():
>
>
http://explain.so/explain.php?uri=3Dhttps://raw.github.com/ircmaxell/PHPPHP=
/master/php.php
>
>         explain() has a problem in copying [printing] (some) literal
constants that I've not yet been able to trace, so didn't want to introduce
the same bug here ...
>
>         No RFC exists, I'd be happy to create one at some point if we're
discussing inclusion, and it's necessary for one to exist ...

Awesome work. There is definitively a need for such tool.

One thought, I wonder how hard it is to integrate it in an IDE, if there
are specials featured which could make ide integrations smoother or nicer.

I would love to have it ad soon as possible on the core. However I'd to be
extremely careful for the 1st step. It would be very good to define a set
of must have features, make them rock solid, bundle the new sapi, and then
continue the development, as usual.

The reason to choose this way is to avoid what happened with opcache and
5.5, delayed final release but still unstable opcache.

Thanks again for your great work!

Cheers,
Pierre

--001a11c264a43d48b404eb4d081f--