[PATCH] Combine same ternary types in GIN and TSearch

Поиск
Список
Период
Сортировка
От Pavel Borisov
Тема [PATCH] Combine same ternary types in GIN and TSearch
Дата
Msg-id CALT9ZEHVMrbbnps7Rame2xShnh96Bfk+bfTXR7tVYXOUE1=esw@mail.gmail.com
обсуждение исходный текст
Ответы Re: [PATCH] Combine same ternary types in GIN and TSearch
Список pgsql-hackers
Hi, hackers!

For historical reasons, now we have two differently named but similar ternary data types in TSearch and Gin text-related types. Before v13 there was also Gin's private TS_execute() version, from which we eventually shifted to Tsearch's TS_execute(). 

To make things more even and beautiful I've made a minor refactor to combine two left ternary types into one.

<gin.h>
typedef char GinTernaryValue
#define GIN_FALSE 0
#define GIN_TRUE 1
#define GIN_MAYBE 2

<ts_utils.h>
typedef enum { TS_NO, TS_YES, TS_MAYBE } TSTernaryValue;

The change is simple and most of it is just the text replacement. The only thing worth noting is that some code does pointer cast between *bool and *TernaryValue so the size of them should coincide. (Declaration done in char type because simple enum on most architectures will be of int size). There is no actual change in the code despite the order of header files inclusion in some modules.

What do you think about this?

--
Best regards,
Pavel Borisov

Postgres Professional: http://postgrespro.com
Вложения

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

Предыдущее
От: Amit Kapila
Дата:
Сообщение: Re: Parallel copy
Следующее
От: Greg Nancarrow
Дата:
Сообщение: Re: Parallel INSERT (INTO ... SELECT ...)