Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:73360
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.212.175 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAO6UHD40eL2eW9T_1mtTnnxcBavSkV51nZ_uTLLc9nWYCbxo=w@mail.gmail.com>
References: <CAO6UHD40eL2eW9T_1mtTnnxcBavSkV51nZ_uTLLc9nWYCbxo=w@mail.gmail.com>
Date: Sat, 22 Mar 2014 11:04:56 -0400
Message-ID: <CAL+t6cHOiBro=PUJnrA_n_iP_96KuXnrsVFX10ymONMPiAJW2g@mail.gmail.com>
To: Chad Scribner <cwscribner@gmail.com>
Cc: PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=e89a8f2357434f7eda04f53353d7
Subject: Re: [PHP-DEV] Pull Request #595
From: theanomaly.is@gmail.com (Sherif Ramadan)

--e89a8f2357434f7eda04f53353d7
Content-Type: text/plain; charset=ISO-8859-1

On Fri, Mar 21, 2014 at 10:29 PM, Chad Scribner <cwscribner@gmail.com>wrote:

> Hello All,
>
> I'm here to request that my pull request #595 get merged.  It's a fix for
> bug #52384 regarding PDO::debugDumpParams().  I've put the code and a phpt
> into the PR.  You'll notice in the PR that the Travis CI build failed but
> after investigating, my test did not fail.  It looks as though it was due
> to other code.  This is my first time making a contribution so feel free to
> let me know if there's anything else I need to add.  Thanks!
>
> ~Chad
>


Hey Chad,

I took a look at your PR some time ago, I believe. Sorry, I did not have
the time to merge it last time being so busy with work, but it looks good.
I saw a few useful comments on the PR on github by nikic and datibbaw.
Maybe take those under consideration. I'll be happy to merge it, though, if
no one else does by tomorrow night.

--e89a8f2357434f7eda04f53353d7--