Rename 'lpp' to 'lp' in heapam.c

Поиск
Список
Период
Сортировка
От Yaphters W
Тема Rename 'lpp' to 'lp' in heapam.c
Дата
Msg-id CAHi_D_YPh8Hgs1sK26vruuNjH-zwNK8j0QP4TapieU9HiCU9Vw@mail.gmail.com
обсуждение исходный текст
Ответы Re: Rename 'lpp' to 'lp' in heapam.c  (David Rowley <dgrowleyml@gmail.com>)
Список pgsql-hackers
Дерево обсуждения
Hi,

I just found the naming of the ItemId variables is not consistent in heapam.c. There are 13 'lpp's and 112 'lp's. Technically 'lpp' is correct as ItemId is a line pointer's pointer and there used to be code like "++lpp" for line pointer array iteration. Now that all the "++lpp" code has been removed and there are 100+ more occurrences of 'Ip' than 'lpp', I suggest we change 'lpp' to 'lp' to make things consistent and avoid confusion.

Best Regards,
Zian Wang
Вложения

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

Предыдущее
От: Peter Geoghegan
Дата:
Сообщение: Re: Overhauling "Routine Vacuuming" docs, particularly its handling of freezing
Следующее
От: John Naylor
Дата:
Сообщение: Re: [PATCH] Clarify the behavior of the system when approaching XID wraparound