Re: [COMMITTERS] pgsql: Generate fmgr prototypes automatically

Поиск
Список
Период
Сортировка
От Peter Eisentraut
Тема Re: [COMMITTERS] pgsql: Generate fmgr prototypes automatically
Дата
Msg-id 40b53c62-4cf6-2d79-4aaa-c76a76532caa@2ndquadrant.com
обсуждение исходный текст
Ответ на Re: [COMMITTERS] pgsql: Generate fmgr prototypes automatically  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: [COMMITTERS] pgsql: Generate fmgr prototypes automatically  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-committers
On 1/17/17 2:35 PM, Tom Lane wrote:
> Peter Eisentraut <peter_e@gmx.net> writes:
>> Generate fmgr prototypes automatically
>
> BTW, now that I've looked through this patch ... why does it add
>
> +#include "nodes/nodes.h"
> +#include "nodes/pg_list.h"
>
> to utils/builtins.h?  AFAICS that shouldn't be necessary, since
> there are no declarations in builtins.h that weren't there before.
> And seeing that this results in builtins.h being included in even more
> places than before, surely we want to minimize that file's inclusion
> footprint.

There are things in builtins.h that technically need those declarations.
 It seems to work without it now, but at some point during the
refactoring they were necessary.  I can remove them again.

--
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


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

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: [COMMITTERS] pgsql: Generate fmgr prototypes automatically
Следующее
От: Alvaro Herrera
Дата:
Сообщение: [COMMITTERS] pgsql: Fix typo