diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2023-11-07 19:26:39 +0100 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2023-11-07 19:26:39 +0100 |
| commit | 615f5f6faa0fb659c4e20e1d2137a869cd33058b (patch) | |
| tree | 0da308badec8c8b7232400724d06897e763b593c /src/backend/utils/adt/windowfuncs.c | |
| parent | 3c551ebede46194237f82062b54b92e474b5c743 (diff) | |
| download | postgresql-615f5f6faa0fb659c4e20e1d2137a869cd33058b.tar.gz postgresql-615f5f6faa0fb659c4e20e1d2137a869cd33058b.zip | |
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 <tgl@sss.pgh.pa.us>
Discussion: https://postgr.es/m/202311071106.6y7b2ascqjlz@alvherre.pgsql
Diffstat (limited to 'src/backend/utils/adt/windowfuncs.c')
| -rw-r--r-- | src/backend/utils/adt/windowfuncs.c | 1 |
1 files changed, 1 insertions, 0 deletions
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" |