Re: Command Triggers - Mailing list pgsql-hackers

From Dimitri Fontaine
Subject Re: Command Triggers
Date
Msg-id m2ipj6sbxy.fsf@2ndQuadrant.fr
Whole thread Raw
In response to Re: Command Triggers  (Alvaro Herrera <alvherre@commandprompt.com>)
List pgsql-hackers
Alvaro Herrera <alvherre@commandprompt.com> writes:
> I didn't like the new cmdtrigger.h file.  It's included by a lot of
> other headers, and it's also itself including execnodes.h and
> parsenodes.h which means practically the whole lot of the source tree
> is included in turn.  If you could split it, so that the struct
> definition is in a different file that's only included by the few .c
> files that actually use that struct, I'd be much happier.

I didn't realize that, thanks for reviewing!

> ... after looking at it more closely, I think only this line needs to be
> in a separate file:
>
> typedef struct CommandContextData *CommandContext;
>
> and that file is included by other headers; they don't need the function
> declarations or the struct definition.

I'll look into that tomorrow then. The same trick is already applied to
Relation and RelationData (resp. in src/include/utils/relcache.h and
src/include/utils/rel.h), and only now I understand why :)

Regards,
--
Dimitri Fontaine
http://2ndQuadrant.fr     PostgreSQL : Expertise, Formation et Support


pgsql-hackers by date:

Previous
From: Dimitri Fontaine
Date:
Subject: Re: Command Triggers
Next
From: Guillaume Lelarge
Date:
Subject: Bug in intarray?