It's not broken because the header file is included via other headers,
but for better style we should be more explicit.
Reported-by: [email protected]
This is the source code of the trigger function:
#include "postgres.h"
+#include "fmgr.h"
#include "executor/spi.h" /* this is what you need to work with SPI */
#include "commands/trigger.h" /* ... triggers ... */
#include "utils/rel.h" /* ... and relations */