Re: Replace magic numbers with strategy numbers for B-tree indexes

Поиск
Список
Период
Сортировка
От Dilip Kumar
Тема Re: Replace magic numbers with strategy numbers for B-tree indexes
Дата
Msg-id CAFiTN-tmJMcUAqvCOHRJnT8L3bdGE8MMPohmnQ1rabp4=s0OSA@mail.gmail.com
обсуждение исходный текст
Список pgsql-hackers
Дерево обсуждения 
On Mon, Jun 30, 2025 at 8:51 AM Daniil Davydov <3danissimo@gmail.com> wrote:
>
> Hi,
> I noticed that some asserts and cycles use magic numbers 1 and 0
> instead of BTLessStrategyNumber and InvalidStrategy.
> At the same time, the BTMaxStrategyNumber macro is used there.
> I suggest using appropriate macros for 1 and 0 values.
>
> Please, see attached patch (targeted on the master branch).

IMHO, it makes sense to use macros when it's already present for
consistency.  So +1 or making this change and the attached patch LGTM


--
Regards,
Dilip Kumar
Google

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