Re: Use MySQL syntax only for parsing MySQL statements in PDO

From: Date: Sun, 11 Apr 2021 17:48:59 +0000
Subject: Re: Use MySQL syntax only for parsing MySQL statements in PDO
References: 1 2 3 4  Groups: php.internals 
Request: Send a blank email to internals+get-114022@lists.php.net to get a copy of this message
Kamil,

> You are attempting to fix a small issue in another driver while the MySQL
parser is already broken with similar bugs.

Yes, this is what the original idea was given the existing bugs on
bugs.php.net while I wasn't aware of all the other MySQL parser problems
that you pointed out.

> Which is kind of my point.

Point taken.

Matteo,

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?

> 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).

Regards,
Sergei Morozov


Thread (7 messages)

« previous php.internals (#114022) next »