<div dir="ltr">Hello Marc,<div><br></div><div>`Parser::$STATEMENT_PARSERS` is a constant (its value remains the same during run-time), but I could not declare it as I should because it is illegal to declare constant arrays in versions prior to PHP 5.6. Also, I decided to avoid the `const` keyword in integer or string constants for consistency reasons. Do you think that I should rename the constant arrays and add the `const` keyword to other declarations?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 8, 2015 at 3:14 PM, Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Dan,<br>
<br>
At this point, since you prefer to work in your own repository [0], I<br>
cannot add line comments via Github so I'll send my comments to this list.<br>
<br>
Please explain why in [1] you are using uppercase. This should be only<br>
for constants.<br>
<br>
<br>
[0] <a href="https://github.com/udan11/sql-parser" target="_blank">https://github.com/udan11/sql-parser</a><br>
<br>
[1] <a href="https://github.com/udan11/sql-parser/blob/master/src/Parser.php#L19" target="_blank">https://github.com/udan11/sql-parser/blob/master/src/Parser.php#L19</a><br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Marc Delisle | phpMyAdmin<br>
<br>
------------------------------------------------------------------------------<br>
_______________________________________________<br>
Phpmyadmin-devel mailing list<br>
<a href="mailto:Phpmyadmin-devel@lists.sourceforge.net">Phpmyadmin-devel@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel" target="_blank">https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel</a><br>
</font></span></blockquote></div><br></div>