From 615f5f6faa0fb659c4e20e1d2137a869cd33058b Mon Sep 17 00:00:00 2001 From: Alvaro Herrera Date: Tue, 7 Nov 2023 19:26:39 +0100 Subject: Stop including parsenodes.h in plannodes.h I added it by mistake in commit 7103ebb7aae8. To clean up, struct MergeAction needs to be moved to primnodes.h from parsenodes.h. (This forces us to also move OverridingKind to primnodes.h). Having to add parsenodes.h to bootstrap.h as fallout is a bit surprising, since nothing nominally needs it there. However, per comments in bootscanner.l, it is needed so that YYSTYPE can be declared. I think this only started with commit dac048f71ebb, but I didn't actually verify that. In passing, stop including parsenodes.h in tcopprot.h. Nothing needs it there. Per discussion on a patch by Ashutosh Bapat. Reviewed-by: Tom Lane Discussion: https://postgr.es/m/202311071106.6y7b2ascqjlz@alvherre.pgsql --- src/backend/utils/adt/windowfuncs.c | 1 + 1 file changed, 1 insertion(+) (limited to 'src/backend/utils/adt/windowfuncs.c') diff --git a/src/backend/utils/adt/windowfuncs.c b/src/backend/utils/adt/windowfuncs.c index b87a624fb2f..0bfbac00d7b 100644 --- a/src/backend/utils/adt/windowfuncs.c +++ b/src/backend/utils/adt/windowfuncs.c @@ -13,6 +13,7 @@ */ #include "postgres.h" +#include "nodes/parsenodes.h" #include "nodes/supportnodes.h" #include "utils/builtins.h" #include "windowapi.h" -- cgit v1.2.3