Обсуждение: unfathomable comment in psqlscan.l

Поиск
Список
Период
Сортировка

unfathomable comment in psqlscan.l

От
Robert Haas
Дата:
While reviewing Pavel's patch for variable quoting in psql, I came
upon the following comment in psqlscan.l:
       /*        * The variable value is just emitted without any        * further examination.  This is consistent
withthe        * pre-8.0 code behavior, if not with the way that        * variables are handled outside backslash
commands.       */
 

I cannot figure out for the life of me what this is referring to.  It
seems to date back to 2004, when the old psql lexer was rewritten to
use flex.  But I can't figure out what "further examination" someone
might have expected, or how it's inconsistent with the how things are
"handled outside backslash commands".

Help?

...Robert


Re: unfathomable comment in psqlscan.l

От
Tom Lane
Дата:
Robert Haas <robertmhaas@gmail.com> writes:
> While reviewing Pavel's patch for variable quoting in psql, I came
> upon the following comment in psqlscan.l:

>         /*
>          * The variable value is just emitted without any
>          * further examination.  This is consistent with the
>          * pre-8.0 code behavior, if not with the way that
>          * variables are handled outside backslash commands.
>          */

> I cannot figure out for the life of me what this is referring to.

At the other place where :foo is expanded, the value is pushed for
re-scanning; thus for example ":bar" within the variable value
might be further expanded as a reference to some other variable.
Inside backslash commands we don't do that but just take the value
literally.
        regards, tom lane


Re: unfathomable comment in psqlscan.l

От
Robert Haas
Дата:
On Tue, Jan 26, 2010 at 2:12 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Robert Haas <robertmhaas@gmail.com> writes:
>> While reviewing Pavel's patch for variable quoting in psql, I came
>> upon the following comment in psqlscan.l:
>
>>         /*
>>          * The variable value is just emitted without any
>>          * further examination.  This is consistent with the
>>          * pre-8.0 code behavior, if not with the way that
>>          * variables are handled outside backslash commands.
>>          */
>
>> I cannot figure out for the life of me what this is referring to.
>
> At the other place where :foo is expanded, the value is pushed for
> re-scanning; thus for example ":bar" within the variable value
> might be further expanded as a reference to some other variable.
> Inside backslash commands we don't do that but just take the value
> literally.

Ah, makes sense.  Thanks.

...Robert