Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:114024
Message-ID: <3af0d306-05fd-c0a1-28c7-06efc885d428@beccati.com>
Date: Mon, 12 Apr 2021 08:00:06 +0200
MIME-Version: 1.0
Content-Language: en-GB
To: Sergei Morozov <morozov@tut.by>, Kamil Tekiela <tekiela246@gmail.com>
Cc: PHP Internals <internals@lists.php.net>
References: <CAP-hcNSGz6WQaCaebLdDm26emTrzxBUpyYvO0MF6anVB8t+=mg@mail.gmail.com>
 <CAGBsUrexRQiCuFbecaGGv_uho9m+QeUfO9+uG4A5z2beyZ1Sog@mail.gmail.com>
 <CAP-hcNSUWtLfiHc_VcGCPC=uneA8BCE=qNLp6p-S8niP_kmaQA@mail.gmail.com>
 <CAGBsUrfhTFuH3Q2ckTwT=z1hZqNTF8eL6kOreSQp1Lb8fHBikg@mail.gmail.com>
 <CAP-hcNS5f=Ow5jfvocwtP9dvT9vjxcD_2u_Rs0xTZX4fcbEOcg@mail.gmail.com>
In-Reply-To: <CAP-hcNS5f=Ow5jfvocwtP9dvT9vjxcD_2u_Rs0xTZX4fcbEOcg@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] Use MySQL syntax only for parsing MySQL statements in
 PDO
From: php@beccati.com (Matteo Beccati)

Sergei,

On 11/04/2021 19:48, Sergei Morozov wrote:
> This is great to see this effort happening! As I understand, you also
> implement an API for platform-specific parsers but via a much better
> extensible API. >
> I'm curious if instead of keeping the MySQL syntax in the default parser
> and creating one for PostgreSQL, it would make sense to create a MySQL
> parser and remove the support for backslash escaping from the default one?

That's a possibility too. I'll think about it.


>> Perhaps we could work together and come up with an RFC?
> 
> Thanks for the offer but I don't think I'll be able to prioritize this
> effort over my other projects. Based on the conversation above, reworking
> the parser the right way requires a lot of investment and frankly speaking
> it's not the most critical problem in the world to solve (all these bugs
> are edge cases).

Fair enough. Any other volunteer, I can bribe promising a phpday 
elephpant in exchange ;-)


Cheers
-- 
Matteo Beccati

Development & Consulting - http://www.beccati.com/