Re: [HACKERS] proposal: psql command \graw

Поиск
Список
Период
Сортировка
От Fabien COELHO
Тема Re: [HACKERS] proposal: psql command \graw
Дата
Msg-id alpine.DEB.2.20.1711092047030.4821@lancre
обсуждение исходный текст
Ответ на Re: [HACKERS] proposal: psql command \graw  (Pavel Stehule <pavel.stehule@gmail.com>)
Ответы Re: [HACKERS] proposal: psql command \graw
Список pgsql-hackers
Hello Pavel,

> I hope so I fixed all mentioned issues.

Patch applies with a warning:
 > git apply ~/psql-graw-2.patch /home/fabien/psql-graw-2.patch:192: new blank line at EOF. + warning: 1 line adds
whitespaceerrors.
 

Otherwise it compiles. "make check" ok. doc gen ok.

Two spurious empty lines are added before StoreQueryTuple.

Doc: "If <literal>+</literal> is appended to the command name, a column 
names are displayed."

I suggest instead: "When <literal>+</literal> is appended, column names 
are also displayed."

ISTM that you can remove "force_column_header" and just set "tuple_only" 
to what you need, that is you do not need to change anything in function 
"print_unaligned_text".

-- 
Fabien.


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

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

Предыдущее
От: Stephen Frost
Дата:
Сообщение: Re: [HACKERS] Simplify ACL handling for large objects and removal ofsuperuser() checks
Следующее
От: Paul Ramsey
Дата:
Сообщение: [HACKERS] Inlining functions with "expensive" parameters