Skip to content

Commit d575051

Browse files
peterepetere
committed
Reformat some node comments
Reformat some comments in node field definitions to avoid long lines. This makes room for per-field annotations. Similar to 835d476. Reviewed-by: Andrew Dunstan <andrew@dunslane.net> Discussion: https://www.postgresql.org/message-id/flat/4b27fc50-8cd6-46f5-ab20-88dbaadca645@eisentraut.org
1 parent 1e1eb12 commit d575051

File tree

1 file changed

+57
-29
lines changed

1 file changed

+57
-29
lines changed

‎src/include/nodes/parsenodes.h

+57-29
Original file line numberDiff line numberDiff line change
@@ -1067,18 +1067,26 @@ typedef struct RangeTblEntry
10671067
* relation. This allows plans referencing AFTER trigger transition
10681068
* tables to be invalidated if the underlying table is altered.
10691069
*/
1070-
Oid relid; /* OID of the relation */
1071-
bool inh; /* inheritance requested? */
1072-
char relkind; /* relation kind (see pg_class.relkind) */
1073-
int rellockmode; /* lock level that query requires on the rel */
1074-
Index perminfoindex; /* index of RTEPermissionInfo entry, or 0 */
1075-
struct TableSampleClause *tablesample; /* sampling info, or NULL */
1070+
/* OID of the relation */
1071+
Oid relid;
1072+
/* inheritance requested? */
1073+
bool inh;
1074+
/* relation kind (see pg_class.relkind) */
1075+
char relkind;
1076+
/* lock level that query requires on the rel */
1077+
int rellockmode;
1078+
/* index of RTEPermissionInfo entry, or 0 */
1079+
Index perminfoindex;
1080+
/* sampling info, or NULL */
1081+
struct TableSampleClause *tablesample;
10761082

10771083
/*
10781084
* Fields valid for a subquery RTE (else NULL):
10791085
*/
1080-
Query *subquery; /* the sub-query */
1081-
bool security_barrier; /* is from security_barrier view? */
1086+
/* the sub-query */
1087+
Query *subquery;
1088+
/* is from security_barrier view? */
1089+
bool security_barrier;
10821090

10831091
/*
10841092
* Fields valid for a join RTE (else NULL/zero):
@@ -1123,11 +1131,15 @@ typedef struct RangeTblEntry
11231131
* merged columns could not be dropped); this is not accounted for in
11241132
* joinleftcols/joinrighttcols.
11251133
*/
1126-
JoinType jointype; /* type of join */
1127-
int joinmergedcols; /* number of merged (JOIN USING) columns */
1128-
List *joinaliasvars; /* list of alias-var expansions */
1129-
List *joinleftcols; /* left-side input column numbers */
1130-
List *joinrightcols; /* right-side input column numbers */
1134+
JoinType jointype;
1135+
/* number of merged (JOIN USING) columns */
1136+
int joinmergedcols;
1137+
/* list of alias-var expansions */
1138+
List *joinaliasvars;
1139+
/* left-side input column numbers */
1140+
List *joinleftcols;
1141+
/* right-side input column numbers */
1142+
List *joinrightcols;
11311143

11321144
/*
11331145
* join_using_alias is an alias clause attached directly to JOIN/USING. It
@@ -1144,8 +1156,10 @@ typedef struct RangeTblEntry
11441156
* implicit, and must be accounted for "by hand" in places such as
11451157
* expandRTE().
11461158
*/
1147-
List *functions; /* list of RangeTblFunction nodes */
1148-
bool funcordinality; /* is this called WITH ORDINALITY? */
1159+
/* list of RangeTblFunction nodes */
1160+
List *functions;
1161+
/* is this called WITH ORDINALITY? */
1162+
bool funcordinality;
11491163

11501164
/*
11511165
* Fields valid for a TableFunc RTE (else NULL):
@@ -1155,14 +1169,18 @@ typedef struct RangeTblEntry
11551169
/*
11561170
* Fields valid for a values RTE (else NIL):
11571171
*/
1158-
List *values_lists; /* list of expression lists */
1172+
/* list of expression lists */
1173+
List *values_lists;
11591174

11601175
/*
11611176
* Fields valid for a CTE RTE (else NULL/zero):
11621177
*/
1163-
char *ctename; /* name of the WITH list item */
1164-
Index ctelevelsup; /* number of query levels up */
1165-
bool self_reference; /* is this a recursive self-reference? */
1178+
/* name of the WITH list item */
1179+
char *ctename;
1180+
/* number of query levels up */
1181+
Index ctelevelsup;
1182+
/* is this a recursive self-reference? */
1183+
bool self_reference;
11661184

11671185
/*
11681186
* Fields valid for CTE, VALUES, ENR, and TableFunc RTEs (else NIL):
@@ -1182,24 +1200,34 @@ typedef struct RangeTblEntry
11821200
* all three lists (as well as an empty-string entry in eref). Testing
11831201
* for zero coltype is the standard way to detect a dropped column.
11841202
*/
1185-
List *coltypes; /* OID list of column type OIDs */
1186-
List *coltypmods; /* integer list of column typmods */
1187-
List *colcollations; /* OID list of column collation OIDs */
1203+
/* OID list of column type OIDs */
1204+
List *coltypes;
1205+
/* integer list of column typmods */
1206+
List *coltypmods;
1207+
/* OID list of column collation OIDs */
1208+
List *colcollations;
11881209

11891210
/*
11901211
* Fields valid for ENR RTEs (else NULL/zero):
11911212
*/
1192-
char *enrname; /* name of ephemeral named relation */
1193-
Cardinality enrtuples; /* estimated or actual from caller */
1213+
/* name of ephemeral named relation */
1214+
char *enrname;
1215+
/* estimated or actual from caller */
1216+
Cardinality enrtuples;
11941217

11951218
/*
11961219
* Fields valid in all RTEs:
11971220
*/
1198-
Alias *alias; /* user-written alias clause, if any */
1199-
Alias *eref; /* expanded reference names */
1200-
bool lateral; /* was LATERAL specified? */
1201-
bool inFromCl; /* present in FROM clause? */
1202-
List *securityQuals; /* security barrier quals to apply, if any */
1221+
/* user-written alias clause, if any */
1222+
Alias *alias;
1223+
/* expanded reference names */
1224+
Alias *eref;
1225+
/* was LATERAL specified? */
1226+
bool lateral;
1227+
/* present in FROM clause? */
1228+
bool inFromCl;
1229+
/* security barrier quals to apply, if any */
1230+
List *securityQuals;
12031231
} RangeTblEntry;
12041232

12051233
/*

0 commit comments

Comments
 (0)