search

forward / sql-parser

A SQL parser written in pure JS
MIT License
531 stars 132 forks source link

Regex for literals should allow upper-case letters. #9

Closed dtimm closed 9 years ago

dtimm commented 9 years ago

It's far enough down the list that it should mess with the other tokens.

andykent commented 9 years ago

I don't think this is needed.

The /i on the regex should mean that it is already case insensitive but if you are able to provide a test case that fails without this change then I will gladly take another look.

dtimm commented 9 years ago

You are correct... I'm trying to sort out a somewhat ridiculous query and seeing why it won't parse.