Re: indentation in _hash_pgaddtup()

Поиск
Список
Период
Сортировка
От Ted Yu
Тема Re: indentation in _hash_pgaddtup()
Дата
Msg-id CALte62wCJ1N8t7On8LeJfoaDrH72bqPOVvqX-hcC_ut_ynbypg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: indentation in _hash_pgaddtup()  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: indentation in _hash_pgaddtup()  (David Rowley <dgrowleyml@gmail.com>)
Список pgsql-hackers


On Thu, Nov 24, 2022 at 12:31 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
David Rowley <dgrowleyml@gmail.com> writes:
> After running pgindent on v2, I see it still pushes the lines out
> quite far. If I add a new line after PageGetItemId(page, and put the
> variable assignment away from the variable declaration then it looks a
> bit better. It's still 1 char over the limit.

If you wanted to be hard-nosed about 80 character width, you could
pull out the PageGetItemId call into a separate local variable.
I wasn't going to be quite that picky, but I won't object if that
seems better to you.

                        regards, tom lane

Patch v4 stores ItemId in a local variable. 
Вложения

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

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: indentation in _hash_pgaddtup()
Следующее
От: Peter Smith
Дата:
Сообщение: Re: Time delayed LR (WAS Re: logical replication restrictions)