Re: Replace offnum++ by OffsetNumberNext

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: Replace offnum++ by OffsetNumberNext
Дата
Msg-id 8309.1207343063@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Replace offnum++ by OffsetNumberNext  (Fujii Masao <fujii.masao@oss.ntt.co.jp>)
Ответы Re: Replace offnum++ by OffsetNumberNext  (Bruce Momjian <bruce@momjian.us>)
Список pgsql-patches
Fujii Masao <fujii.masao@oss.ntt.co.jp> writes:
> This is the patch replace offnum++ by OffsetNumberNext.
> According to off.h, OffsetNumberNext is the macro prepared to
> disambiguate the different manipulations on OffsetNumbers.
> But, increment operator was used in some places instead of the macro.

I wonder if we shouldn't go the other way, ie, use ++ everywhere.
OffsetNumberNext seems like just useless obscurantism ...

            regards, tom lane

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

Предыдущее
От: Teodor Sigaev
Дата:
Сообщение: Partial match in GIN
Следующее
От: "Pavel Stehule"
Дата:
Сообщение: fixing sql generated features - plpgsql case statement