Re: Add Boolean node

Поиск
Список
Период
Сортировка
От Zhihong Yu
Тема Re: Add Boolean node
Дата
Msg-id CALNJ-vT2Rz+4GfnJgm60sqN07sg2H7Q31FLXqQ8tnf8aDonF6A@mail.gmail.com
обсуждение исходный текст
Ответ на Add Boolean node  (Peter Eisentraut <peter.eisentraut@enterprisedb.com>)
Список pgsql-hackers
Hi,
For buildDefItem():

+       if (strcmp(val, "true") == 0)
+           return makeDefElem(pstrdup(name),
+                              (Node *) makeBoolean(true),
+                              -1);
+       if (strcmp(val, "false") == 0)

Should 'TRUE' / 'FALSE' be considered above ?

-       issuper = intVal(dissuper->arg) != 0;
+       issuper = boolVal(dissuper->arg) != 0;

Can the above be written as (since issuper is a bool):

+       issuper = boolVal(dissuper->arg);

Cheers

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

Предыдущее
От: Bharath Rupireddy
Дата:
Сообщение: Re: Add checkpoint and redo LSN to LogCheckpointEnd log message
Следующее
От: Fujii Masao
Дата:
Сообщение: Re: Allow escape in application_name