Re: [PATCH] Cleanup of PLpgSQL_recfield

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: [PATCH] Cleanup of PLpgSQL_recfield
Дата
Msg-id 1050.1225843708@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: [PATCH] Cleanup of PLpgSQL_recfield  ("Jonah H. Harris" <jonah.harris@gmail.com>)
Список pgsql-hackers
"Jonah H. Harris" <jonah.harris@gmail.com> writes:
> On Tue, Nov 4, 2008 at 3:57 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> I am not real sure why the code is inconsistent about spelling the
>> second field's name differently in some of the structs, but it seems
>> like a bad idea --- as you've demonstrated, it invites confusion.
>> What would probably be better is a patch to rename exprno, rfno, etc
>> to all be called dno to make this connection more obvious.

> Attached.  Passed regressions and basic testing.

Looks good, applied.
        regards, tom lane


В списке pgsql-hackers по дате отправления:

Предыдущее
От: Emmanuel Cecchet
Дата:
Сообщение: Re: Transactions and temp tables
Следующее
От: Guillaume Lelarge
Дата:
Сообщение: Re: Patch for ALTER DATABASE WITH TABLESPACE