Re: Patch for Re: [HACKERS] Bug of PL/pgSQL parser - Mailing list pgsql-bugs

From Tom Lane
Subject Re: Patch for Re: [HACKERS] Bug of PL/pgSQL parser
Date
Msg-id 17904.1028056851@sss.pgh.pa.us
Whole thread Raw
In response to Re: Patch for Re: [HACKERS] Bug of PL/pgSQL parser  (Bruce Momjian <pgman@candle.pha.pa.us>)
Responses Re: Patch for Re: [HACKERS] Bug of PL/pgSQL parser
List pgsql-bugs
Bruce Momjian <pgman@candle.pha.pa.us> writes:
> I assume this completes the TODO item:
>         o Fix PL/PgSQL to handle quoted mixed-case identifiers

It would if it worked, but it doesn't --- for one thing, it seems
to allow empty identifiers (surely {DQUOTE}* should be {DQUOTE}+).
I'm not convinced it parses identifiers the same as the main scanner
does anyway.  Why not just lift the flex code for identifiers
out of parser/scan.l?

            regards, tom lane

pgsql-bugs by date:

Previous
From: Tom Lane
Date:
Subject: Re: Bug #724: Problmes creating aggregate functions in 7.2.1
Next
From: Bruce Momjian
Date:
Subject: Re: Patch for Re: [HACKERS] Bug of PL/pgSQL parser