diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2021-02-02 14:35:12 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2021-02-02 14:35:25 -0500 |
| commit | 78fab843386bbaaaa4f74b26859d669e89d7db63 (patch) | |
| tree | 962fc77aec0e9f04d63b10d33975f05ea24f4e24 | |
| parent | 7428469586e837747f86566c790f6ccceac1b1f3 (diff) | |
| download | postgresql-78fab843386bbaaaa4f74b26859d669e89d7db63.tar.gz postgresql-78fab843386bbaaaa4f74b26859d669e89d7db63.zip | |
Remove extra increment of plpgsql's statement counter for FOR loops.
This left gaps in the internal statement numbering, which is not
terribly harmful (else we'd have noticed sooner), but it's not
great either.
Oversight in bbd5c207b; backpatch to v12 where that came in.
Pavel Stehule
Discussion: https://postgr.es/m/CAFj8pRDXyQaJmpotNTQVc-t-WxdWZC35V2PnmwOaV1-taidFWA@mail.gmail.com
| -rw-r--r-- | src/pl/plpgsql/src/pl_gram.y | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/pl/plpgsql/src/pl_gram.y b/src/pl/plpgsql/src/pl_gram.y index af63fe126f9..12f75ed2bb9 100644 --- a/src/pl/plpgsql/src/pl_gram.y +++ b/src/pl/plpgsql/src/pl_gram.y @@ -1304,7 +1304,6 @@ stmt_for : opt_loop_label K_FOR for_control loop_body new = (PLpgSQL_stmt_fori *) $3; new->lineno = plpgsql_location_to_lineno(@2); - new->stmtid = ++plpgsql_curr_compile->nstatements; new->label = $1; new->body = $4.stmts; $$ = (PLpgSQL_stmt *) new; @@ -1319,7 +1318,6 @@ stmt_for : opt_loop_label K_FOR for_control loop_body /* forq is the common supertype of all three */ new = (PLpgSQL_stmt_forq *) $3; new->lineno = plpgsql_location_to_lineno(@2); - new->stmtid = ++plpgsql_curr_compile->nstatements; new->label = $1; new->body = $4.stmts; $$ = (PLpgSQL_stmt *) new; |