8000 Properly mark rules that return no value. · linearregression/postgres@e136986 · GitHub
[go: up one dir, main page]

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit e136986

Browse files
committed
Properly mark rules that return no value.
1 parent 5fa77c2 commit e136986

File tree

1 file changed

+11
-17
lines changed

1 file changed

+11
-17
lines changed

src/backend/parser/gram.y

Lines changed: 11 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
*
1212
*
1313
* IDENTIFICATION
14-
* $Header: /cvsroot/pgsql/src/backend/parser/gram.y,v 2.330 2002/06/18 17:56:41 momjian Exp $
14+
* $Header: /cvsroot/pgsql/src/backend/parser/gram.y,v 2.331 2002/06/19 15:40:58 momjian Exp $
1515
*
1616
* HISTORY
1717
* AUTHOR DATE MAJOR EVENT
@@ -161,7 +161,6 @@ static void doNegateFloat(Value *v);
161161

162162
%type <list> createdb_opt_list
163163
%type <defelt> createdb_opt_item
164-
%type <boolean> opt_equal
165164

166165
%type <ival> opt_lock, lock_type
167166
%type <boolean> opt_force, opt_or_replace
@@ -224,12 +223,10 @@ static void doNegateFloat(Value *v);
224223
%type <defelt> createfunc_opt_item
225224
%type <typnam> func_arg, func_return, func_type, aggr_argtype
226225

227-
%type <boolean> opt_arg, TriggerForOpt, TriggerForType, OptTemp, OptWithOids
226+
%type <boolean> opt_arg, TriggerForType, OptTemp, OptWithOids
228227

229228
%type <list> for_update_clause, opt_for_update_clause, update_list
230229
%type <boolean> opt_all
231-
%type <boolean> opt_table
232-
%type <boolean> opt_chain
233230

234231
%type <node> join_outer, join_qual
235232
%type <jtype> join_type
@@ -310,8 +307,6 @@ static void doNegateFloat(Value *v);
310307
%type <list> constraints_set_list
311308
%type <boolean> constraints_set_mode
312309

313-
%type <boolean> opt_as
314-
315310

316311
/*
317312
* If you make any token changes, update the keyword table in
@@ -2044,8 +2039,8 @@ TriggerForSpec:
20442039
;
20452040

20462041
TriggerForOpt:
2047-
EACH { $$ = TRUE; }
2048-
| /*EMPTY*/ { $$ = FALSE; }
2042+
EACH {}
2043+
| /*EMPTY*/ {}
20492044
;
20502045

20512046
TriggerForType:
@@ -3360,15 +3355,14 @@ opt_trans: WORK {}
33603355
| /*EMPTY*/ {}
33613356
;
33623357

3363-
opt_chain: AND NO CHAIN { $$ = FALSE; }
3358+
opt_chain: AND NO CHAIN {}
33643359
| AND CHAIN
33653360
{
33663361
/* SQL99 asks that conforming dbs reject AND CHAIN
33673362
* if they don't support it. So we can't just ignore it.
33683363
* - thomas 2000-08-06
33693364
*/
33703365
elog(ERROR, "COMMIT / CHAIN not yet supported");
3371-
$$ = TRUE;
33723366
}
33733367
;
33743368

@@ -3511,8 +3505,8 @@ createdb_opt_item:
35113505
* equals for backward compability, and it doesn't seem worth removing it.
35123506
* 2002-02-25
35133507
*/
3514-
opt_equal: '=' { $$ = TRUE; }
3515-
| /*EMPTY*/ { $$ = FALSE; }
3508+
opt_equal: '=' {}
3509+
| /*EMPTY*/ {}
35163510
;
35173511

35183512

@@ -3578,8 +3572,8 @@ CreateDomainStmt:
35783572
}
35793573
;
35803574

3581-
opt_as: AS {$$ = TRUE; }
3582-
| /* EMPTY */ {$$ = FALSE; }
3575+
opt_as: AS {}
3576+
| /* EMPTY */ {}
35833577
;
35843578

35853579

@@ -4088,8 +4082,8 @@ OptTempTableName:
40884082
}
40894083
;
40904084

4091-
opt_table: TABLE { $$ = TRUE; }
4092-
| /*EMPTY*/ { $$ = FALSE; }
4085+
opt_table: TABLE {}
4086+
| /*EMPTY*/ {}
40934087
;
40944088

40954089
opt_all: ALL { $$ = TRUE; }

0 commit comments

Comments
 (0)
0